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

show server in fuzzy search #303

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Evidlo
Copy link
Contributor

@Evidlo Evidlo commented Aug 6, 2021

preview

I was also trying to implement this on the room list pane on the left, but it seems like it doesn't implement the SetDynamicColors function. Any ideas?

@tulir
Copy link
Owner

tulir commented Sep 7, 2021

This doesn't seem to define Room.GetServer(), and showing the server is somewhat questionable anyway (since there's no "the server" in Matrix).

Showing bridge info protocol or network displayname for single-bridge rooms that were created by the bridge bot might be a better solution (less generic, but more technically correct). All libera portal rooms probably have bridge info at least.

@Evidlo
Copy link
Contributor Author

Evidlo commented Dec 13, 2021

I added the missing function.

and showing the server is somewhat questionable anyway (since there's no "the server" in Matrix)

It may still be useful to show the domain as entered by the user (i.e. the room alias). Better than having identical entries in the switcher.

@Evidlo
Copy link
Contributor Author

Evidlo commented Apr 7, 2022

There's a Main Address field in Element room settings. I'm not sure what property in the spec it maps to, but wouldn't it make sense to put that information here?

out

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

Successfully merging this pull request may close these issues.

2 participants