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

Some action columns could be rows

Open M123-dev opened this issue 1 year ago • 9 comments

What

Image

Screenshot_20220909-104230_OpenFoodFacts

Yes and no would easily fit into a row, but maybe not on every language a automatic align mode would be perfect

M123-dev avatar Sep 09 '22 14:09 M123-dev

I noticed there's no border in 'No' Button. Can we add border to make both buttons look similar, in terms of UI?

anshpathania7 avatar Sep 12 '22 13:09 anshpathania7

@anshpathania7 Possibly linked to #2988.

I guess the idea would be to have similar displays for negative and positive action, except that background for positive is full, and background for negative is empty but with a border (same color as positive full background).

That would make negative buttons more explicit, especially when the context is a bit confusing.

monsieurtanuki avatar Sep 12 '22 13:09 monsieurtanuki

Okay, I checked the code. Structure of buttons are decided on the basis of screen size currently, it can be turned into row if screen is large or if Axis is provided explicitly. So, for this issue, in automatic align mode should we also consider, string length of languages?

anshpathania7 avatar Sep 12 '22 13:09 anshpathania7

String length wouldn't be enough:

  • IIIIIIIIII
  • OOOOOOOOOO

Actual display size should be computed instead. How do they do in AutoSizeText to check the display length and decide to use a smaller font?

monsieurtanuki avatar Sep 12 '22 16:09 monsieurtanuki

Seems like this was already addressed?

image

WildOrangutan avatar Oct 01 '22 18:10 WildOrangutan

Seems like this was already addressed.

image

Should this be used to be designed like this?

prathamsoni11 avatar Oct 04 '22 05:10 prathamsoni11

It's interesting that this doesn't happen on your devices. I still have this problem in 3.20.0+740 (lastest merged pr from yesterday), probably again something which is depending on the screen size

M123-dev avatar Oct 04 '22 07:10 M123-dev

I'm able to reproduce it if I increase font and display scale on my device. Will try to address it.

WildOrangutan avatar Oct 04 '22 17:10 WildOrangutan

Perfect @WildOrangutan I don't know how it's currently solved or what the problem exactly is, but the OverflowBar widget seems promising

M123-dev avatar Oct 04 '22 18:10 M123-dev

Can I work on this issue?

prathamsoni11 avatar Nov 06 '22 05:11 prathamsoni11

Sure @prathamsoni11 I guess the best way to try to reproduce this is using the new Android rezisable emulator or the DevicePreview we have in debug builds

M123-dev avatar Nov 06 '22 12:11 M123-dev