Skip to content

Fix scrollbar apperance #34065

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
Merged
merged 6 commits into from
Sep 15, 2022
Merged

Fix scrollbar apperance #34065

merged 6 commits into from
Sep 15, 2022

Conversation

@dartcafe
Copy link
Contributor

@dartcafe dartcafe commented Sep 14, 2022 •
edited
Loading

fix #34064

Before
grafik
After
grafik

related #33736

@dartcafe
Fix scrollbar apperance
571a85a 
fix #34064 

Signed-off-by: René Gieling <github@dartcafe.de>
@dartcafe dartcafe added bug design Design, UI, UX, etc. 3. to review Waiting for reviews feature: scss labels Sep 14, 2022
@dartcafe dartcafe requested a review from skjnldsv September 14, 2022 06:22
@dartcafe dartcafe added this to the Nextcloud 25 milestone Sep 14, 2022
@skjnldsv
Copy link
Member

Screenshots please? :)

szaimen
szaimen reviewed Sep 14, 2022
View reviewed changes
@szaimen szaimen added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Sep 14, 2022
@dartcafe
scss and compile
f1d35a1
@dartcafe
Fix scrollbar apperance
93d3ca3 
fix #34064

Signed-off-by: René Gieling <github@dartcafe.de>
Signed-off-by: dartcafe <github@dartcafe.de>
@dartcafe
scss and compile
2fc1f4b 
Signed-off-by: dartcafe <github@dartcafe.de>
@dartcafe
Merge branch 'bug/noid/main-scrollbar' of https://github.com/nextclou…
f1881f2 
…d/server into pr-34065/nextcloud/bug/noid/main-scrollbar
@dartcafe
server.scss
63d5156
@dartcafe dartcafe added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 15, 2022
jancborchardt
jancborchardt approved these changes Sep 15, 2022
View reviewed changes
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Good catch! :)

PVince81
PVince81 approved these changes Sep 15, 2022
View reviewed changes
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@PVince81 PVince81 merged commit 4f244e9 into master Sep 15, 2022
@PVince81 PVince81 deleted the bug/noid/main-scrollbar branch September 15, 2022 12:04
@skjnldsv skjnldsv mentioned this pull request Sep 15, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@szaimen szaimen szaimen left review comments

@PVince81 PVince81 PVince81 approved these changes

@jancborchardt jancborchardt jancborchardt approved these changes

@skjnldsv skjnldsv Awaiting requested review from skjnldsv

Assignees

No one assigned

Labels

3. to review Waiting for reviews bug design Design, UI, UX, etc. feature: scss

Projects

None yet

Milestone

Nextcloud 25

Development

Successfully merging this pull request may close these issues.

[NC25] Scrollbar of main section is not of style thin

6 participants

@dartcafe @skjnldsv @PVince81 @jancborchardt @szaimen