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

"R" logo when searching for users is ugly #2934

Closed
ara4n opened this issue Jan 12, 2017 · 3 comments · Fixed by matrix-org/matrix-react-sdk#732
Closed

"R" logo when searching for users is ugly #2934

ara4n opened this issue Jan 12, 2017 · 3 comments · Fixed by matrix-org/matrix-react-sdk#732
Assignees
@lukebarnard1
Labels
P2 S-Tolerable Low/no impact on users
Milestone
UX Fixes

Comments

@ara4n
Copy link
Member

ara4n commented Jan 12, 2017

I think we should remove it for now, until we support searching user directories exposed by bridges, which we don't yet.
@ara4n ara4n modified the milestones: ui, UX Fixes Jan 12, 2017
@ara4n ara4n mentioned this issue Jan 12, 2017
Open
@ara4n ara4n added S-Tolerable Low/no impact on users P2 ui/ux and removed P2 labels Jan 14, 2017
@ara4n ara4n self-assigned this Jan 16, 2017
@dbkr dbkr mentioned this issue Jan 18, 2017
Merged
@dbkr dbkr closed this as completed Jan 18, 2017
@lukebarnard1
Copy link
Contributor

lukebarnard1 commented Feb 22, 2017 •
edited
Loading

@dbkr is this really fixed? The "R" is still visible... Surely we could use the "initial" avatars instead?

@ara4n
Copy link
Member Author

ara4n commented Feb 22, 2017

yeah, i'm not sure why this has been closed.

@ara4n ara4n reopened this Feb 22, 2017
@tessgadwa tessgadwa mentioned this issue Feb 22, 2017
Closed
71 tasks
@ara4n
Copy link
Member Author

ara4n commented Feb 22, 2017

oh, i guess it's because the double icon got fixed. but rather than ugly R's we should be showing appropriate avatars instead.

@lukebarnard1 lukebarnard1 mentioned this issue Feb 23, 2017
Closed
@ara4n ara4n assigned lukebarnard1 and unassigned ara4n Mar 2, 2017
@lukebarnard1 lukebarnard1 mentioned this issue Mar 10, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lukebarnard1 lukebarnard1

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

Successfully merging a pull request may close this issue.

Test to see how fuse feels matrix-org/matrix-react-sdk
3 participants
@dbkr @ara4n @lukebarnard1