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

The instance list needs a sort menu #537

Closed
opusforlife2 opened this issue Nov 18, 2022 · 6 comments
Closed

The instance list needs a sort menu #537

opusforlife2 opened this issue Nov 18, 2022 · 6 comments
Labels
Improvement Improvements that can be made to existing feature

Comments

@opusforlife2
Copy link

Some services have such a huge number of instances that it makes it difficult to make sense of the whole thing. A sort menu would really help here.

Some options:

  • Alphabetical order (instance name)
  • Alphabetical order (country of the instance)
  • Presence/absence of Cloudflare CDN
  • Latency (after test done)
@IkelAtomig IkelAtomig added the Improvement Improvements that can be made to existing feature label Nov 25, 2022
@ManeraKai
Copy link
Member

Alphabetical order (country of the instance)

This requires more information from the instances fetcher.

@opusforlife2
Copy link
Author

Ideally, this info should be provided by upstream, I think? It would be pretty inefficient to have to ping instances to find out their countries. But I'm not sure if all services' instance APIs provide country info.

@IkelAtomig
Copy link
Member

Maybe we could sort by Cloudflare and Non-Cloudflare as we have that data already fetched by the Instance fetcher. Possibly Alphabetical too.

But will not be sorted by latency.

ManeraKai added a commit that referenced this issue Dec 7, 2022
@opusforlife2
Copy link
Author

Thanks for adding the feature!

But will not be sorted by latency.

The option could be greyed out until the user manually performs a latency test? From what I've seen, the extension permanently saves that info until the test is done again.

@ManeraKai
Copy link
Member

No, bc we removed Latency Test.

@opusforlife2
Copy link
Author

@ManeraKai Now that the latency test has been re-added, could this be reconsidered?

CatCoder52 added a commit to CatCoder52/extension_browser that referenced this issue Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Improvements that can be made to existing feature
Development

No branches or pull requests

3 participants