Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ style: Enhance Styling & Accessibility #5956

Merged
merged 2 commits into from
Feb 20, 2025
Merged

Conversation

berry-13
Copy link
Collaborator

Summary

Description:

  • 🎨 UI Improvements: Enhanced visual appeal and layout adjustments

    • Added shadows to div elements in various components for a better look
    • Auto-focus enabled for TextareaAutosize in PromptEditor
    • Conditional button margin adjustments in Groups/List
  • Accessibility Improvements: Elevating user access needs

    • Enhanced the SelectDropDown in ModelPanel with aria attributes for better accessibility
  • 🔄 Code Refactoring: Streamlining and optimizing the codebase

    • Refactored SelectDropDown using hooks for improved component functionality and readability
    • Removed redundant code and streamlined formatting
    • Added ring style
  • 📊 Data Model Updates: Minor corrections for consistency

    • Adjusted label and value fields in Categories for proper data handling

These updates ensure a more cohesive user experience, enhanced accessibility features, and cleaner, more maintainable code

Change Type

  • a11y
  • style

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • My changes do not introduce new warnings
  • Local unit tests pass with my changes
  • Any changes dependent on mine have been merged and published in downstream modules.

@berry-13 berry-13 added ♿ a11y Accessibility 🎨 design UI/UX improvements labels Feb 20, 2025
@danny-avila danny-avila changed the title ✨ feat(UI): Enhance Styling & Accessibility ✨ style: Enhance Styling & Accessibility Feb 20, 2025
@danny-avila danny-avila merged commit fe70135 into main Feb 20, 2025
7 checks passed
@danny-avila danny-avila deleted the fix/a11y-issues branch February 20, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♿ a11y Accessibility 🎨 design UI/UX improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants