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

[Bug Fix] ScrollBar width #309

Merged
merged 5 commits into from
Jan 28, 2024
Merged

[Bug Fix] ScrollBar width #309

merged 5 commits into from
Jan 28, 2024

Conversation

PiervitoCreanza
Copy link
Contributor

Hi,
I think I found a bug in the ScrollBar component.
In the image below, notice that the flex-1 class assigned to the ScrollAreaThumb causes the scrollbar to expand and fill the entire width of the scroll area.
immagine

Removing the specified class produces the expected behaviour:
immagine

@sadeghbarati
Copy link
Collaborator

Hi @PiervitoCreanza

Looks like we are missing flex-col in orientation === "horizontal"

Can you turn back flex-1 and add flex-col to the ScrollAreaScrollbar Component

@PiervitoCreanza
Copy link
Contributor Author

Oh, you're right. I reverted my old commits and pushed the new one.

@sadeghbarati sadeghbarati merged commit 8bd5525 into unovue:dev Jan 28, 2024
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