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

3PID lookup when searching for users should give UI feedback if a 3PID is recognised #2933

Closed
2 tasks
ara4n opened this issue Jan 12, 2017 · 0 comments
Closed
2 tasks

3PID lookup when searching for users should give UI feedback if a 3PID is recognised #2933

ara4n opened this issue Jan 12, 2017 · 0 comments
Assignees
@dbkr
Labels
P1 S-Tolerable Low/no impact on users T-Enhancement
Milestone
UX Fixes

Comments

@ara4n
Copy link
Member

ara4n commented Jan 12, 2017

If I search for "matthew@wherever.com" when inviting users, I get absolutely zero UI feedback to imply that i've typed in a valid email address which has matched a 3PID lookup and will actually work. Instead, it feels as if i'm typing into the void - just searching randomly, and no results are turning up.

Things we could to to help this:

  • List "invite matthew@wherever.com by email" as the first item in the search list, assuming a syntactically valid email address has been entered. This doesn't help with recognising when a bound email has been entered though
  • format the entry text differently, or slap a [m] logo or something on the first search result item, if the entry is recognised as a valid 3pid.
@ara4n ara4n added this to the UX Fixes milestone Jan 12, 2017
@ara4n ara4n added S-Tolerable Low/no impact on users feature P1 labels Jan 14, 2017
@ara4n ara4n assigned giomfo and dbkr and unassigned giomfo Jan 16, 2017
dbkr added a commit to matrix-org/matrix-react-sdk that referenced this issue Jan 18, 2017
@dbkr
Better feedback in invite dialog
de62190 
Show feedback if you enter a valid but unknown email address
or mxid

Fixes element-hq/element-web#2933
@dbkr dbkr mentioned this issue Jan 18, 2017
Merged
@dbkr dbkr closed this as completed Jan 26, 2017
@t3chguy t3chguy mentioned this issue Sep 26, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dbkr dbkr

Labels
P1 S-Tolerable Low/no impact on users T-Enhancement
Projects
None yet
Milestone
UX Fixes
Development

No branches or pull requests
4 participants
@jryans @dbkr @ara4n @giomfo