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

browsing directories on other servers is super-unintuitive #5194

Closed
richvdh opened this issue Oct 10, 2017 · 10 comments · Fixed by matrix-org/matrix-react-sdk#4209
Closed

browsing directories on other servers is super-unintuitive #5194

richvdh opened this issue Oct 10, 2017 · 10 comments · Fixed by matrix-org/matrix-react-sdk#4209

Comments

@richvdh
Copy link
Member

richvdh commented Oct 10, 2017

If you're not on matrix.org, you have to type into the dropdown which doesn't look like you should be able to type things into.

Apart from the utter lack of discoverability, you have to type! Words! with your fingers! The brutality!

@richvdh
Copy link
Member Author

richvdh commented Oct 10, 2017

actually apparently you don't have to type it - you can add entries to the dropdown via the config.json for the riot-web instance as per https://github.com/vector-im/riot-web/blob/2b7ee756f8945c933050b6c3a451340241c9bab2/config.sample.json#L9.

But if you want to use one that's not on the list, it's deeply undiscoverable.

@ara4n
Copy link
Member

ara4n commented Oct 10, 2017

also related: #2054

@pafcu
Copy link
Contributor

pafcu commented Oct 11, 2017

It would perhaps be more clear if the placeholder text was something other than "matrix.org". Perhaps "Type a remote server name here".

@richvdh
Copy link
Member Author

richvdh commented Oct 11, 2017

@pafcu: to be honest, I don't think most people get as far as seeing that placeholder text, since you have to focus the dropdown to see it.

@pafcu
Copy link
Contributor

pafcu commented Oct 11, 2017

@richvdh Certainly, but it is an easy step in the right direction.

@lampholder lampholder added T-Defect S-Minor Impairs non-critical functionality or suitable workarounds exist P1 ui/ux labels Oct 11, 2017
@r3pek
Copy link

r3pek commented Oct 25, 2017

I would just remove it altogether. that combo box makes no sense at all. one would just type the domain and be done with it.

@Half-Shot
Copy link
Member

Not sure how it was before, but now there is a blinking cursor in the combo box which seems like a good indicator to me that you can type in it.

@lampholder doesn't seem like a p1 to me, but idk.

@aaronraimist
Copy link
Collaborator

Related: #12495

@jryans jryans added 🔥 Fire 🔥 and removed S-Minor Impairs non-critical functionality or suitable workarounds exist labels Feb 25, 2020
@turt2live
Copy link
Member

@vector-im/design this is your honorary comment asking to have this looked at (as I hear, you've been nudged elsewhere)

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.

10 participants