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

New scrollbars wrong size #11666

Open
Tragen opened this issue Jan 16, 2025 · 3 comments
Open

New scrollbars wrong size #11666

Tragen opened this issue Jan 16, 2025 · 3 comments
Labels
bug It's a bug

Comments

@Tragen
Copy link

Tragen commented Jan 16, 2025

Operating system

Windows

Joplin version

3.2.11

Desktop version info

Joplin 3.2.11 (prod, win32)

Client ID: 80a7bce693094dbe1111f91c81a991db
Sync Version: 3
Profile Version: 47
Keychain Supported: No

Revision: 2fba101

Backup: 1.4.2
Cursor Sync: 2.1.0
Freehand Drawing: 2.14.0
Note overview: 1.7.1

Current behaviour

I've set the scrollbars to small.
The Editor scrollbar is wider than the viewer scrollbar.

Expected behaviour

The viewer scrollbar is too small, nearly not visible.

Editor Scrollbar:
Image

Viewer Scrollbar:
Image

Logs

No response

@Tragen Tragen added the bug It's a bug label Jan 16, 2025
@personalizedrefrigerator
Copy link
Collaborator

Thank you for the feedback!

Be aware that the "scrollbar size" setting currently configures the scrollbar width, not the scrollbar height. By default, with the "small" size, scrollbars should continue to be 7px wide, as was the case with the default app styles before v3.2 (see main.scss in v3.1, noteStyle.ts in v3.1).

Screenshots: Comparing the editor and viewer scrollbar width

To double-check that the scrollbars are the same width (at least on my Linux computer), I stacked screenshots of the scrollbar in the viewer above a screenshot of the scrollbar in the editor:

screenshot: Shows the viewer scrollbar above the editor scrollbar

@Tragen
Copy link
Author

Tragen commented Jan 17, 2025

I talk about the width. If you compare my screenshots, the viewer width is too small and in dark mode it's not really visible

personalizedrefrigerator added a commit to personalizedrefrigerator/joplin that referenced this issue Jan 21, 2025
**Note**: Related to an issue mentioned in laurent22#11666
@personalizedrefrigerator
Copy link
Collaborator

personalizedrefrigerator commented Jan 22, 2025

Related: #11708

Note: The note viewer scrollbar has a different border-radius from the main app scrollbars, possibly causing a slightly smaller width for very short scrollbars. In addition to the contrast issues, this should be resolved by #11708.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug
Projects
None yet
Development

No branches or pull requests

2 participants