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

[front] - chore(sidebarmenu): polish #8920

Merged
merged 7 commits into from
Nov 27, 2024
Merged

[front] - chore(sidebarmenu): polish #8920

merged 7 commits into from
Nov 27, 2024

Conversation

JulesBelveze
Copy link
Contributor

Description

This PR polishes the SidebarMenu to adjust padding and add a shadow effect to scrollable content when disappearing under the div contain the buttons.

Screenshot 2024-11-26 at 17 30 55

Risk

Low

Deploy Plan

Deploy front

 - Remove left padding from the conversation sidebar container for a cleaner layout
 - Add horizontal padding within the scroll area for better content spacing and readability
…in SidebarMenu

- Consolidated action buttons for multi-select mode to improve accessibility and interface consistency
- Added a shadow box style to enhance visual separation of action buttons from conversation list
…ering of sidebar menu actions

 - Simplified the conditionals by separating the multi-select and default action views into distinct blocks
 - Removed redundant classes and elements to clean up the UI structure for better maintainability
@JulesBelveze JulesBelveze requested a review from tdraier November 26, 2024 16:31
…port from SidebarMenu

 - Cleanup of SidebarMenu component by removing the import statement for Separator which is no longer used
 - Replace inline shadow styles with a `shadow-fade` utility class for consistency and maintainability
 - Apply the new `shadow-fade` class to elements within the `SidebarMenu.tsx` file
…tyling for SidebarMenu

- Replace `shadow-fade` with `shadow-tale` in SidebarMenu for multi-select and action buttons
- Remove deprecated `shadow-fade` styling from global.css

[front/tailwind.config] - feature: add new 'tale' box-shadow to tailwind configuration

- Introduce a new `tale` box-shadow definition to the TailwindCSS configuration for future use in styling
Copy link
Contributor

@tdraier tdraier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice !

 - Ensure consistent ordering of CSS classes for improved readability and adherence to styling guidelines
 - Modify the positioning of 'shadow-tale' to be the last class in the class list for sidebar elements
@JulesBelveze JulesBelveze merged commit 477cc17 into main Nov 27, 2024
3 checks passed
@JulesBelveze JulesBelveze deleted the fix/sidebarmenu branch November 27, 2024 08:56
overmode pushed a commit that referenced this pull request Nov 27, 2024
* [front] - fix: adjust conversation sidebar padding and scroll area

 - Remove left padding from the conversation sidebar container for a cleaner layout
 - Add horizontal padding within the scroll area for better content spacing and readability

* [assistant/conversation] - refactor: streamline multi-select actions in SidebarMenu

- Consolidated action buttons for multi-select mode to improve accessibility and interface consistency
- Added a shadow box style to enhance visual separation of action buttons from conversation list

* [front/components/assistant/conversation] - refactor: streamline rendering of sidebar menu actions

 - Simplified the conditionals by separating the multi-select and default action views into distinct blocks
 - Removed redundant classes and elements to clean up the UI structure for better maintainability

* [front/assistant/conversation] - refactor: remove unused Separator import from SidebarMenu

 - Cleanup of SidebarMenu component by removing the import statement for Separator which is no longer used

* [front] - refactor: streamline shadow styling in SidebarMenu components

 - Replace inline shadow styles with a `shadow-fade` utility class for consistency and maintainability
 - Apply the new `shadow-fade` class to elements within the `SidebarMenu.tsx` file

* [front/components/assistant/conversation] - refactor: update shadow styling for SidebarMenu

- Replace `shadow-fade` with `shadow-tale` in SidebarMenu for multi-select and action buttons
- Remove deprecated `shadow-fade` styling from global.css

[front/tailwind.config] - feature: add new 'tale' box-shadow to tailwind configuration

- Introduce a new `tale` box-shadow definition to the TailwindCSS configuration for future use in styling

* [front] - fix: adjust shadow-tale CSS class ordering in SidebarMenu

 - Ensure consistent ordering of CSS classes for improved readability and adherence to styling guidelines
 - Modify the positioning of 'shadow-tale' to be the last class in the class list for sidebar elements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants