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

letters in key preview popups doesn't fit #712

Open
alekksander opened this issue Apr 16, 2024 · 3 comments · Fixed by #744
Open

letters in key preview popups doesn't fit #712

alekksander opened this issue Apr 16, 2024 · 3 comments · Fixed by #744
Labels
bug Something isn't working low priority This issue is unlikely to be resolved in the near future due to maintainers' priorities

Comments

@alekksander
Copy link

Letters displayed in key popups are in disproportion. This happens since forked. For example P and G are misaligned, but Letter M seems is elongated.

Screenshot_20240416-094250_Mull
Screenshot_20240416-095317_Mull
Screenshot_20240416-095408_Mull

@alekksander alekksander added the bug Something isn't working label Apr 16, 2024
@BabyOilJohnson
Copy link

P and G issue is already reported here #166

Which version are u using?

@alekksander
Copy link
Author

Latest beta (1.3), but as mentioned this is observable since "openboard"

@Helium314
Copy link
Owner

I'll re-open this, as the width issue is not fixed with #744.
In short: letter size depends on screen width in pixels, but the width of the popup also depends on the DisplayMetrics.density of the device, getting smaller for higher densities.

@Helium314 Helium314 reopened this Apr 25, 2024
@Helium314 Helium314 added the low priority This issue is unlikely to be resolved in the near future due to maintainers' priorities label Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working low priority This issue is unlikely to be resolved in the near future due to maintainers' priorities
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants