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.

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

Make the shields the same in Element Web and Element Android #52

Closed
Linerly opened this issue Oct 25, 2021 · 6 comments · Fixed by element-hq/element-android#4338
Closed

Comments

@Linerly
Copy link

Linerly commented Oct 25, 2021

Is your other session showing the same shield?

The first time I verify a login against another session was confusing: Element Web and Element Android have a different shield with the checkmark (in other means, I took it literally), but I knew it has the checkmark in both Element Web and Element Android, so I click "Yes"

@Linerly Linerly changed the title Make the sields the same in Element Web and Element Android Make the shields the same in Element Web and Element Android Oct 25, 2021
@BillCarsonFr
Copy link
Member

FTR web shield vs android shield
image

So they are different. Notice that other clients would probably use another icon (as it's not part of the spec)

Maybe we should re-think the copy?

@kittykat
Copy link
Contributor

@BillCarsonFr : There is more difference with the Android dark theme

@BillCarsonFr
Copy link
Member

Current copy > "Almost there! Is the other device showing the same shield?"

@BillCarsonFr
Copy link
Member

BillCarsonFr commented Oct 25, 2021

@BillCarsonFr : There is more difference with the Android dark theme

yeah there is probably a white border
image

@BillCarsonFr
Copy link
Member

Where can I find the web icon sources? can't find it in figma?

@aaronraimist
Copy link

The web issue for this is #1887 and the suggestion from M was to change the copy (in addition to just changing the icon)

We should probably change the prompt to be "Is the other user showing a tick?" in order to be more cross-client compatible, or something rather than "the same shield".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants