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

Presence indicator centered over profile photo #5489

Closed
thefirstofthe300 opened this issue Mar 9, 2022 · 3 comments · Fixed by #5604
Closed

Presence indicator centered over profile photo #5489

thefirstofthe300 opened this issue Mar 9, 2022 · 3 comments · Fixed by #5604
Assignees
@ouchadam
Labels
A-Toolbars O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect Something isn't working: bugs, crashes, hangs and other reported problems X-Regression Z-WTF WTF moment: High Impact, Low Effort
Projects
Issue triage Android App Team

Comments

@thefirstofthe300
Copy link
Contributor

Steps to reproduce

markup_89 (1)

The presence indicator is covering up the profile photo.

Outcome

What did you expect?

Presence indicator should be located in the bottom right.

Your phone model

Pixel 6 Pro

Operating system version

12

Application version and app store

Element v1.4.4

Homeserver

Synapse 1.53

Will you send logs?

No
@thefirstofthe300 thefirstofthe300 added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Mar 9, 2022
@RiotRobot RiotRobot added this to Incoming in Issue triage Mar 9, 2022
@binarydad
Copy link

Same here, multiple android versions.

@ouchadam ouchadam moved this from Incoming to Triaged in Issue triage Mar 11, 2022
@ouchadam ouchadam added S-Tolerable Low/no impact on users O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-Toolbars Z-WTF WTF moment: High Impact, Low Effort X-Regression labels Mar 11, 2022
@notramo
Copy link
Contributor

notramo commented Mar 18, 2022

Only in 1.4.4. In 1.4.2 it worked fine.

@notramo notramo mentioned this issue Mar 20, 2022
Closed
@notramo
Copy link
Contributor

notramo commented Mar 20, 2022

I noticed that when the presence indicator is centered, the encryption indicator shield is displayed in the bottom right corner of the avatar, where the presence dot should be.
image
However, it didn't show up immediately, only after a few minutes of using it, and cache reloads. Possibly related with #5584

@ouchadam ouchadam self-assigned this Mar 22, 2022
@ouchadam ouchadam added this to In progress in Android App Team Mar 22, 2022
@ouchadam ouchadam mentioned this issue Mar 22, 2022
Merged
6 tasks
Issue triage automation moved this from Triaged to Closed Mar 22, 2022
Android App Team automation moved this from In progress to Merged Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ouchadam ouchadam

Labels
A-Toolbars O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect Something isn't working: bugs, crashes, hangs and other reported problems X-Regression Z-WTF WTF moment: High Impact, Low Effort
Projects
Android App Team
Merged
Issue triage
Closed
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixing presence icon anchoring to the middle of the room icon element-hq/element-android
4 participants
@binarydad @ouchadam @thefirstofthe300 @notramo