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

[HxSidebar] responsive collapsing #943

Merged
merged 7 commits into from
Nov 19, 2024
Merged

Conversation

crdo
Copy link
Member

@crdo crdo commented Nov 19, 2024

Fixes #282 and #532.

What i did to fix #532
To hide the item content when the sidebar is collapsed, instead of conditional rendering with if statement, i am adding css utility classes that only hide the content on viewport larger than Sidebars ResponsiveBreakpoint setting and if resized below, it gets displayed again.

@hakenr I need two things from you to be able to complete this PR:

  1. Review the API of HxSidebarItemNavLinkContent (naming, etc.)
  2. Extract GetResponsiveCssClass somewhere so it is not duplicated (HxSidebar, HxSidebarItem, HxSidebarBrand) and it can be used on other places as I find it useful.

I have also overhauled Sidebars CSS vars so you can create color variations with utility classes and added respective demos and added flexbox layout to implement what was requested in #282.

@crdo crdo requested a review from hakenr November 19, 2024 07:44
@hakenr hakenr changed the title Feature/sidebar responsive collapsing [HxSidebar] responsive collapsing Nov 19, 2024
@hakenr hakenr merged commit 97d1bb8 into master Nov 19, 2024
1 check passed
@hakenr hakenr deleted the feature/SidebarResponsiveCollapsing branch November 19, 2024 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants