app-center icon indicating copy to clipboard operation
app-center copied to clipboard

Some parts of App Center do not scale text correctly with "Large Text" mode enabled

Open shytzedaka opened this issue 2 years ago • 15 comments

When using the accessibility mode "Large Text" as presented here: image or using the terminal command to get the same effect: gsettings set org.gnome.desktop.interface text-scaling-factor 1.25

Text does not appear to be extending or scaling in some areas of the App Center as can be seen here: image

it appears the issue only occurs at the sidebar of the app center and not the rest of the program. Note: I am Using a 1920x1080 Display with regular 100% scale and the App Center is fully maximized image

shytzedaka avatar Sep 25 '23 11:09 shytzedaka

Thanks for the screenshots! We definitely need to improve the spacing in some places to accommodate larger fonts. A few more screenshots (with ratings backend running):

Normal Font Large Font
Screenshot from 2023-09-25 13-56-22 Screenshot from 2023-09-25 13-56-14
Screenshot from 2023-09-25 13-57-48 Screenshot from 2023-09-25 13-58-02

@anaserejio: something to look at when you got time :)

d-loose avatar Sep 25 '23 12:09 d-loose

@d-loose I have improved the spacings. Screenshot from 2023-10-06 11-01-21 Screenshot from 2023-10-06 11-01-42 Screenshot from 2023-10-06 11-22-01

Should I send a PR, you check it out?

anorld-droid avatar Oct 06 '23 08:10 anorld-droid

@d-loose I have improved the spacings.

Thanks - looks great! I'm wondering if there's enough space for the ratings though. Can you check what it looks with the ratings enabled?

RATINGS_SERVICE_URL=ratings.ubuntu.com flutter run

d-loose avatar Oct 06 '23 09:10 d-loose

What is the expected outcome for this, should we increase it's width? https://user-images.githubusercontent.com/113362648/270333836-c626f3e7-0a5d-4bf0-b74a-cf223311ba09.png

I don't know yet, we'll need to patiently wait for @anasereijo, who works on the design for the app center.

d-loose avatar Oct 06 '23 09:10 d-loose

@d-loose I have improved the spacings.

Thanks - looks great! I'm wondering if there's enough space for the ratings though. Can you check what it looks with the ratings enabled?

RATINGS_SERVICE_URL=ratings.ubuntu.com flutter run

Yeah, not so good. We may have to increase the height of the card to accommodate the summary;it has a maxlines of 2 OR make its maxlines to 1.

What do you think, @d-loose?

image

anorld-droid avatar Oct 06 '23 09:10 anorld-droid

Hi @shytzedaka, thank you for filling the issue. We are planning on doing an accessibility assessment on the app, but in the meantime, feel free to file any issues or propose improvements. :)

@anorld-droid thanks for doing this 🙌 one question: are you changing the height of the cards when Large text is enabled or for both situations (enabled and disabled)?

anasereijo avatar Oct 06 '23 10:10 anasereijo

Hi @shytzedaka, thank you for filling the issue. We are planning on doing an accessibility assessment on the app, but in the meantime, feel free to file any issues or propose improvements. :)

@anorld-droid thanks for doing this 🙌 one question: are you changing the height of the cards when Large text is enabled or for both situations (enabled and disabled)?

Both, but I will add when Large text is enabled; I had not thought of that.

anorld-droid avatar Oct 06 '23 11:10 anorld-droid

hey! it seems that this bug so far persists in released Ubuntu 23.10.1 (Clean install of the latest public ISO) I'm not sure if this is still being fixed or not but i thought it would be worth pointing that out since we're out of the beta phase.

image

shytzedaka avatar Nov 04 '23 22:11 shytzedaka

Text does not scale with other languages like Russian, where the word meaning Manage ("Управление") or About ("Об App Center") are too big to fit and get cut off by dots.

I don't think it looks right. I'm using the Snap Store from the Edge channel.

olekzonder avatar Nov 26 '23 22:11 olekzonder

Is there an update on this?

shytzedaka avatar May 22 '24 12:05 shytzedaka

We'll have some time for polishing and bug fixing this cycle and also plan to do some refactoring :). While a lot of accessibility issues need to be fixed in the Flutter embedder, this issue is simply caused by an inflexible implementation of the layout and can/should be fixed here.

d-loose avatar May 23 '24 12:05 d-loose

Thank you for your hard work :)

shytzedaka avatar May 23 '24 12:05 shytzedaka

Just want to chime in that even without Large Text mode or any sort of scaling enabled it can't spell "Manage" completely on a 1080p screen even though it looks like there's plenty of space. Just upgraded to 24.04.1 from 22.04.4, not sure if it's something on my end or happens for others as well.

image image

frankhuurman avatar Sep 02 '24 15:09 frankhuurman

Just want to chime in that even without Large Text mode or any sort of scaling enabled it can't spell "Manage" completely on a 1080p screen even though it looks like there's plenty of space. Just upgraded to 24.04.1 from 22.04.4, not sure if it's something on my end or happens for others as well.

That was a different issue, see #1782 (fixed in latest/edge, will be rolled out to stable branches soon)

d-loose avatar Sep 02 '24 15:09 d-loose

@d-loose ah perfect, thanks for the quick reply Dennis :)

frankhuurman avatar Sep 02 '24 18:09 frankhuurman