Page MenuHomePhabricator

[BUG] [REGRESSION] Incorrect font styling on text-fields in the reading list name and description dialog
Closed, ResolvedPublicBUG REPORT

Description

Steps to Reproduce:
  1. Open any article and save to a reading list
  2. Select to "Create a new list" to bring up the reading list name and description dialog
Actual Results:
  • Input text field and labels are showing as font-weight Medium
  • Input text is showing at a larger font-size (20sp?) than expected

image.png (1×1 px, 195 KB)

Expected Results:
  • Font-weight of input text fields and labels should be Regular
  • Input text should be font-size 16sp (as per standard Subhead style)

image.png (640×360 px, 46 KB)

Detected on

Wikipedia 2.7.239-alpha-2018-08-02

Event Timeline

Change 450258 had a related patch set uploaded (by Dbrant; owner: Dbrant):
[apps/android/wikipedia@master] Design tweak: fix style of reading list name/description dialog.

https://gerrit.wikimedia.org/r/450258

Change 450258 merged by jenkins-bot:
[apps/android/wikipedia@master] Design tweak: fix style of reading list name/description dialog.

https://gerrit.wikimedia.org/r/450258

Hi @Dbrant - font sizes and weight are fixed, but the color of inputted text should be the 'Material theme Primary' color once filled (and 'Material theme Disabled/De-emphasised' when hint text)

image.png (552×353 px, 70 KB)

Change 458859 had a related patch set uploaded (by Dbrant; owner: Dbrant):
[apps/android/wikipedia@master] Design tweak: text colors in TextInputDialog.

https://gerrit.wikimedia.org/r/458859

Change 458859 merged by jenkins-bot:
[apps/android/wikipedia@master] Design tweak: text colors in TextInputDialog.

https://gerrit.wikimedia.org/r/458859

cmadeo subscribed.

@Dbrant the colors look great on the description hint text, but I'm now seeing the Reading list title hint text persist instead of disappearing

Screenshot_20181030-172457.png (2×1 px, 563 KB)

Would it be possible for the field to be cleared when the user taps into the Reading list title field or starts typing, without having the example text appear in black or having it be highlighted?

@cmadeo I'm not totally sure what you mean: are you referring to the blue hint text that floats above the reading list title? If so, this is a standard part of Android input fields.

@Dbrant, ah, no I meant the text that is in the text field. It is black and highlighted rather than grey and unhighlighted like the text field below.

@cmadeo Oh sorry, I see what you mean.
So actually, the text "My reading list" is not a hint, but a pre-populated name for the new reading list. If you choose to click OK, it will actually use that title for the new list. Otherwise you can start typing another title, which will automatically wipe out the pre-highlighted default text.
Do we want to get rid of this behavior?

@Dmitry Brant <dbrant@wikimedia.org>, ah! Thanks for the clarification!
Hmm, I'm assuming that this is the same behavior if a user already has a
list called 'My reading list'?

Yes, in that case the prepopulated text will still be "My reading list", and the text box will show a warning that this name already exists.

@Dbrant if it's not too much trouble then, my preference would be to get rid of this behavior and just use placeholder text that disappears on cursor highlight instead.

Change 471030 had a related patch set uploaded (by Dbrant; owner: Dbrant):
[apps/android/wikipedia@master] Don't prepopulate reading list name when creating new list.

https://gerrit.wikimedia.org/r/471030

Change 471030 merged by jenkins-bot:
[apps/android/wikipedia@master] Don't prepopulate reading list name when creating new list.

https://gerrit.wikimedia.org/r/471030

@Dbrant this is looking good, but would it be possible to have the placeholder text still appear when the keyboard is opened (similar to iOS below).

IMG_E5CB36AB729D-1.jpeg (2×1 px, 342 KB)

@cmadeo Unfortunately that's not the standard behavior of the Android text input component. The placeholder ("label") text automatically gets shifted to the top of the field when it receives focus.