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

🎨 feat: enhance Chat Input UI, File Mgmt. UI, Bookmarks a11y #5112

Merged
merged 21 commits into from
Dec 29, 2024

Conversation

berry-13
Copy link
Collaborator

@berry-13 berry-13 commented Dec 27, 2024

Summary

Description:

  • 🎨 Improvements to Chat User Interface:

    • ChatForm.tsx: Added a new state maximizeChatSpace to dynamically control the chat form's width, providing a better user experience by adjusting to different screen sizes
  • 📁 Enhancements to File Attachments and Previews:

    • AttachFile.tsx: Simplified positioning logic for the attachment button, streamlining user interaction
    • DragDropOverlay.tsx: Updated overlay styling to improve visual feedback during drag-and-drop operations, enhancing user engagement
    • FileContainer.tsx: Enhanced styling and accessibility by adding titles, making file browsing more intuitive
    • FileRow.tsx: Improved layout and rendering logic for file rows, with better handling of various file types, particularly images and non-image files
  • 📊 Updates to File Table Component

    • Columns.tsx: Adjusted column definitions and added hover effects to enhance user interaction and responsiveness
    • DataTable.tsx: Refactored state management to use hooks, improving responsiveness on small screens, thereby enhancing user experience on different devices

This PR is dedicated to improving both the user interface and the underlying code architecture of the chat input components, aiming at a seamless user experience and easier code maintainability. The updates introduce feature enhancements and code simplifications to ensure an intuitive and visually consistent file management process

Change Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Style

Testing

  • 📱 Mobile Browsers:

    • Tested on Chrome, Firefox, Samsung Internet Browser, and Brave
  • 🖥️ Desktop Browsers:

    • Tested on Chrome, Firefox, and Brave

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
  • I have written tests demonstrating that my changes are effective or that my feature works
  • Local unit tests pass with my changes
  • Any changes dependent on mine have been merged and published in downstream modules.

@berry-13 berry-13 marked this pull request as ready for review December 28, 2024 01:01
@berry-13 berry-13 marked this pull request as draft December 28, 2024 17:54
@berry-13 berry-13 changed the title 🎨 style: Revamp File Management Components for Enhanced UX and Code Structure 🎨 feat: enhance Chat Input UI and optimize file management components Dec 29, 2024
@berry-13 berry-13 marked this pull request as ready for review December 29, 2024 01:19
berry-13 and others added 7 commits December 29, 2024 14:28
…RIA labels, and correct html role for key presses
…improved props handling and accessibility features
…hareMenu and DropdownPopup components so focus correctly returns to menu item
…ved props handling and accessibility features
@danny-avila danny-avila added the a11y Accessibility label Dec 29, 2024
@danny-avila danny-avila changed the title 🎨 feat: enhance Chat Input UI and optimize file management components 🎨 feat: enhance Chat Input UI, File Mgmt. UI, Bookmarks a11y Dec 29, 2024
@danny-avila danny-avila merged commit cb19216 into main Dec 29, 2024
2 checks passed
@danny-avila danny-avila deleted the style/few-adjustements branch December 29, 2024 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants