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

Make different zoom levels display correctly #7040

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

len-foss
Copy link

@len-foss len-foss commented Oct 6, 2024

First time contributor checklist:

Contributor checklist:

  • My contribution is not related to translations.
  • My commits are in nice logical chunks with good commit messages
  • My changes are rebased on the latest main branch
  • A npm run ready run passes successfully (more about tests here) [well, Optimize markAllCallHistoryReadInConversation is fast fails but I assume it's because you expect developers to have beefy computers]
  • My changes are ready to be shipped to users

Description

On a fresh install, setting the Zoom level to 200% result in unusable preferences settings.
For some reason, it is forbidden to resize the window, while the window size is not a function of the zoom level nor screen size.
Making it resizable is a "free" tweak that makes it reasonably usable.

Furthermore, even at 150% some component just don't display nicely.

I think the screenshots convey the problem.
Screenshot_20241006_121706
This is at 150%
Screenshot_20241006_121748

The about page is also cropped, making the links overflow the window size.

I've also made some tweaks to make the preferences look all right at all default zoom levels (after is on the left, current production build on the right):
Screenshot_20241006_150200

@ayumi-signal ayumi-signal self-assigned this Oct 7, 2024
@len-foss
Copy link
Author

Hello @ayumi-signal
Is there something blocking, or any way I can help?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants