IceCubesApp icon indicating copy to clipboard operation
IceCubesApp copied to clipboard

Remove tab string from iOS app

Open mofmofmofneko opened this issue 2 years ago • 10 comments

Fixed an issue where the display of tab menu names may be corrupted on iOS devices with small screens.

Closes:#654

mofmofmofneko avatar Feb 06 '23 01:02 mofmofmofneko

Instead of always removing the labels, I'd think to have this configurable (show labels with icons) would be a better way. Especially for new users who open the app for the first time, it might not be clear what these icons are without the explanation by the labels.

sothawo avatar Feb 06 '23 07:02 sothawo

Definitely need to be a setting. And I've yet to pull it to see how it look but I hope iOS adapt the image bottom margin then.

Dimillian avatar Feb 06 '23 08:02 Dimillian

I was of that opinion for a while. However, depending on the language and label size, the characters may be cut off and it does not make sense.

I also refer to the fact that the Mastodon official app does not have an icon label by default.

Tab

mofmofmofneko avatar Feb 06 '23 08:02 mofmofmofneko

And what about a screenshots when it's removed? Thanks!

Dimillian avatar Feb 06 '23 08:02 Dimillian

This is a very good example that making this configurable is the best way. For a user where the label is long and who has a small display it can be disabled. But I'd keep the label as default for everybody else.

sothawo avatar Feb 06 '23 08:02 sothawo

Here is a screenshot of the simulator.

Simulator

Explicitly showing functions to users with strings is important, but considering the accessibility of people who use languages ​​that cannot read the default English and have not been translated, UI that can be conveyed only by icons is important.

mofmofmofneko avatar Feb 06 '23 08:02 mofmofmofneko

Having it as an option is the optimal solution. Usability guidelines recommend labels, but we're talking about icons that, in this case, have universally recognized functionality, true pictographs

echidistanti avatar Feb 06 '23 10:02 echidistanti

I like the idea, again. But this create a huge UI problem. The icons should be center aligned, and we can't do that in SwiftUI until we make a custom tabbar. I would be ok to merge it if it's an option in display settings, with a default to true (showing the label). Until then I can't merge it.

Dimillian avatar Feb 06 '23 11:02 Dimillian

I understand your opinion.

Then we plan to find time and make changes to add a show/hide option to the settings screen.

mofmofmofneko avatar Feb 07 '23 07:02 mofmofmofneko

If the name of the setting item that controls the visibility you want is "Show tab name", please do a code review.

mofmofmofneko avatar Feb 09 '23 01:02 mofmofmofneko

All good thanks!

Dimillian avatar Feb 09 '23 05:02 Dimillian