mollyim / mollyim-android

Enhanced and security-focused fork of Signal.
GNU Affero General Public License v3.0
1.58k stars 86 forks source link

Text color needs to be white in "Re-link your devices" prompt on unlinked secondary Molly device with dark theme #256

Open taivlam opened 9 months ago

taivlam commented 9 months ago

Is there an existing issue for this?

Bug description

After issue #219 was resolved, I noticed there needs to be an improvement in the UI: when the message telling the user to re-link the device appears with black text at the bottom of the screen, which is hard to read when Molly's theme is set to black. Please see the photo below.

photo1

(It's hard to tell in the crop, but I had to turn on the brightness all the way up on my Pixel Tablet in order for the black text to be sufficiently legible.)

This may be due to the fact that the Pixel Tablet is using an LCD display, instead of OLED displays that are on the Pixel phones.

Making the text color for the message

Re-link your devices

The devices you added were unlinked when your device was unregistered. Go to Settings to re-link any devices.

would improve the readability, as the buttons for "Open settings" and "Later" are in a lighter color.

If needed, I can upload another photo showing the whole device (but I'd need to block out my contact's info).

Steps to reproduce

  1. Relink pre-existing secondary Molly device to primary Molly device.
  2. See the UI error above.

(At least that's what I think will happen, I haven't relinked my device since.)

Molly version

v6.41.3-2-FOSS

Android version

Android 14 (GrapheneOS version 2023121200)

Device

Google Pixel Tablet (tangorpro)

Link to debug log

No response