Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"x is typing" notification UX issue and flash of light mode #6645

Closed
daniellekirkwood opened this issue Aug 26, 2022 · 2 comments · Fixed by #6665
Closed

"x is typing" notification UX issue and flash of light mode #6645

daniellekirkwood opened this issue Aug 26, 2022 · 2 comments · Fixed by #6665
Assignees
@gileluard
Labels
T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Team: Delight Z-AppLayout

Comments

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Aug 26, 2022 •
edited
Loading

Steps to reproduce

@ara4n raised this issue here

image

The "X is typing" notification has a bug and there's a flicker of light themed composer when in dark theme

Example of the flash:

whiteflash.mov
@daniellekirkwood daniellekirkwood added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Aug 26, 2022
@gileluard gileluard self-assigned this Aug 30, 2022
@gileluard
Copy link
Contributor

@ara4n

I cannot reproduce "x is typing" notification UX issue anymore. May my PR also fixed this? Can you still reproduce it?

@gileluard gileluard mentioned this issue Sep 2, 2022
Merged
@thibaultamartin
Copy link
Contributor

Probably related but I don't know if #6665 fixes:

  • I used element iOS during the night (dark mode)
  • I opened the app in the morning (the phone had turned in light mode before opening element iOS)
  • Most of the UI updated to light mode
  • The "Synchronising…" pill stayed in dark mode

It should be easy to reproduce by:

  • Switching to dark mode
  • Opening the app
  • Putting the app in background
  • Switching to light mode
  • Optionally waiting for few minutes
  • Putting Element in the foreground

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gileluard gileluard

Labels
T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Team: Delight Z-AppLayout
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

UI glitches in dark mode element-hq/element-ios
3 participants
@thibaultamartin @gileluard @daniellekirkwood