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]: In some languages, an extra scrollbar appears in the settings #6330

Open
6 tasks done
MarkusGaugg opened this issue Dec 9, 2024 · 5 comments · May be fixed by #6655
Open
6 tasks done

[Bug]: In some languages, an extra scrollbar appears in the settings #6330

MarkusGaugg opened this issue Dec 9, 2024 · 5 comments · May be fixed by #6655
Labels
B: visual bug Something isn't working good first issue Good for newcomers U: reproduced

Comments

@MarkusGaugg
Copy link
Contributor

Guidelines

  • I have encountered this bug in the latest release of FreeTube.
  • I have encountered this bug in the official downloads of FreeTube.
  • I have searched the issue tracker for open and closed issues that are similar to the bug report I want to file, without success.
  • I have searched the documentation for information that matches the description of the bug I want to file, without success.
  • This issue contains only one bug.

Describe the bug

  1. Open Free Tube v0.22.0-nightly-5283 Beta (latest build)
  2. Set the language to German or Turkish
  3. Open settings and then enable "Sort Settings Selections (A-Z)"
  4. Now there's another scrollbar showing up here - marked in red
  5. Always reproducible - but only in certain languages like German or Turkish.

Ashampoo_Snap_Montag, 9  Dezember 2024_20h33m44s

Expected Behavior

This is how it should look.

Ashampoo_Snap_Montag, 9  Dezember 2024_20h35m50s

Issue Labels

visual bug

FreeTube Version

v0.22.0-nightly-5283 Beta

Operating System Version

Windows 11 - 23H2

Installation Method

.exe

Primary API used

Local API

Last Known Working FreeTube Version (If Any)

No response

Additional Information

No response

Nightly Build

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

efb4f5ff-1298-471a-8973-3d47447115dc commented Dec 9, 2024

Reproduced in v0.22.0-nightly-5283 Beta with the provided steps above.

@MarkusGaugg
Copy link
Contributor Author

This behavior occurs in about half of the languages, including Polish and Russian.

@MarkusGaugg
Copy link
Contributor Author

Because of this bug, a different setting is underlined than the one I actually selected.

Here's a concrete example: I clicked and selected Distraction-Free Mode, and it's showing up, but "General" is still underlined.

Ashampoo_Snap_Montag, 9  Dezember 2024_21h9m54s

@absidue
Copy link
Member

absidue commented Dec 9, 2024

The problem is long tooltip texts, while those texts do exist in other languages too, the alphabetical order of the settings with that toggle enabled, means that in those two languages the sections with the long tooltip texts at the bottom of them end up at the bottom of the list, so instead of overflowing into the other sections they overflow off the bottom of the list, causing the scrollbar to show up.

screenshot

@ChunkyProgrammer ChunkyProgrammer added the good first issue Good for newcomers label Dec 10, 2024
sullivanlng added a commit to sullivanlng/FreeTube that referenced this issue Jan 21, 2025
@sullivanlng
Copy link
Contributor

Hi everyone,

I’ve worked on fixing the issue.
Instead of fully removing the overflow-x: hidden property in the CSS, I’ve decided to comment it out for now. This way, it can easily be reintroduced if needed later.
I tested the fix locally and the scrollbar is no longer there, and everything looks good without introducing any horizontal scroll issues.
Here’s the pull request for review: #6655. Let me know if there’s anything that needs adjusting!

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B: visual bug Something isn't working good first issue Good for newcomers U: reproduced
Projects
Status: To assign
Development

Successfully merging a pull request may close this issue.

5 participants