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.

Sign up for GitHub

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

Jump to bottom

[UI Scrollbars] hide scrollbar in scrollable components #1472

Closed
leiyre opened this issue May 9, 2022 · 0 comments · Fixed by #1490
Closed

[UI Scrollbars] hide scrollbar in scrollable components #1472

leiyre opened this issue May 9, 2022 · 0 comments · Fixed by #1490
Assignees
@leiyre
Labels
area: ui Indicates that an issue or pull request is related to the User Interface (UI) type: bug Indicates an unexpected problem or unintended behavior
Projects
Release
Milestone
v0.15.0

Comments

@leiyre
Copy link
Member

leiyre commented May 9, 2022

The scrollbar is visible in some components (metrics) in all browsers. On the other hand, in some browsers (firefox) they are visible whenever there is scroll.

In general, we can unify and hide the scrollbar in scrollable components without affecting the behavior.
@leiyre leiyre added type: enhancement Indicates new feature requests area: ui Indicates that an issue or pull request is related to the User Interface (UI) labels May 9, 2022
@leiyre leiyre self-assigned this May 9, 2022
@leiyre leiyre added this to Backlog in Release via automation May 9, 2022
@leiyre leiyre moved this from Backlog to In progress in Release May 9, 2022
@frascuchon frascuchon moved this from In progress to Planified in Release May 10, 2022
@frascuchon frascuchon moved this from Planified to In progress in Release May 10, 2022
@frascuchon frascuchon added this to the v0.15.0 milestone May 11, 2022
leiyre added a commit that referenced this issue May 13, 2022
@leiyre
feat(#1472): hide scrollbar in scrollable components
bb3c9e4 
closes #1472

This PR hides the scrollbar in all browsers without affecting the behavior
@leiyre leiyre mentioned this issue May 13, 2022
Merged
@leiyre leiyre linked a pull request May 13, 2022 that will close this issue
feat(#1472): hide scrollbar in scrollable components #1490
Merged
@leiyre leiyre moved this from In progress to Pending Review in Release May 13, 2022
@frascuchon frascuchon moved this from Pending Review to Review OK in Release May 18, 2022
@frascuchon frascuchon added type: bug Indicates an unexpected problem or unintended behavior and removed type: enhancement Indicates new feature requests labels May 18, 2022
Release automation moved this from Review OK to Waiting Release May 18, 2022
frascuchon pushed a commit that referenced this issue May 18, 2022
@leiyre
fix(#1472): hide scrollbar in scrollable components (#1490)
999897a 
This PR hides the scrollbar in all browsers without affecting the behavior

Closes #1472
@frascuchon frascuchon moved this from Waiting Release to Ready to Release QA in Release May 19, 2022
frascuchon pushed a commit that referenced this issue May 19, 2022
@leiyre @frascuchon
fix(#1472): hide scrollbar in scrollable components (#1490)
b056e4e 
This PR hides the scrollbar in all browsers without affecting the behavior

Closes #1472

(cherry picked from commit 999897a)
@leiyre leiyre moved this from Ready to Release QA to Approved Release QA in Release May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leiyre leiyre

Labels
area: ui Indicates that an issue or pull request is related to the User Interface (UI) type: bug Indicates an unexpected problem or unintended behavior
Projects
No open projects
Release
Approved Release QA
Milestone
v0.15.0
Development

Successfully merging a pull request may close this issue.

feat(#1472): hide scrollbar in scrollable components argilla-io/argilla
2 participants
@frascuchon @leiyre