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

improve server browser bot filter icon #2381

Open
hkly-1245 opened this issue Nov 3, 2023 · 5 comments
Open

improve server browser bot filter icon #2381

hkly-1245 opened this issue Nov 3, 2023 · 5 comments
Labels
cat: mod UI Category mod (ui) 💡 Feature New feature or request ❓ Feedback Further information is requested P3: Normal Priority 3
Milestone

Comments

@hkly-1245
Copy link

hkly-1245 commented Nov 3, 2023

Description of the issue (current behavior, expected/desired behavior)

The current version of the server browser features a space invader like icon to filter bots, but with a confusing tooltip about human players.

image

Most icons design, tooltips couple describe accuratly the behavior in this menu, there is a match between them.

Solutions could be:

  • Change the icon with one involving human player representation
    Or
  • Change the behavior of the browser to only keep servers with bots
    Or
  • Change both the icon and the tooltip, with an icon involving both a robot and human representation.

Motivation and use case for changing the behavior

User Interface must be as meaningfull to the user as it can get.

@Aranud Aranud added P3: Normal Priority 3 💡 Feature New feature or request cat: mod UI Category mod (ui) labels Nov 4, 2023
@Aranud Aranud added this to the Future milestone Nov 4, 2023
@hkly-1245
Copy link
Author

I am not a designer, but you get the idea:
image

@winrid
Copy link
Contributor

winrid commented Nov 9, 2023

I think the icon is too small to illustrate a human and a bot. The button has three states, off, bots only, humans only. The tooltip seems pretty descriptive to me.

Probably the text should just be changed. When you select "exclude bots" it says "Don't Show..." Which is confusing.

Instead we can just make it say "Bots Only" or "Humans Only" depending on checkbox state.

@winrid
Copy link
Contributor

winrid commented Nov 15, 2023

Gonna take a stab at this this Sunday

Aranud pushed a commit that referenced this issue Nov 22, 2023
Instead of just showing "include only" or "exclude only", we can show much better short descriptions that still fit fine.
Verified the filters against the vars from the servers for correctness.
@Aranud Aranud modified the milestones: Future, 2.82.0 Nov 22, 2023
@Aranud Aranud added the ❓ Feedback Further information is requested label Nov 22, 2023
@winrid
Copy link
Contributor

winrid commented Nov 25, 2023

@hkly-1245 I think the core issue behind this is resolved? LMK: https://www.youtube.com/watch?v=HErivAFFd_8

@hkly-1245
Copy link
Author

@hkly-1245 I think the core issue behind this is resolved? LMK: https://www.youtube.com/watch?v=HErivAFFd_8

Yes, except for the bot icon graphical design

@Aranud Aranud modified the milestones: 2.82.0, Future Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat: mod UI Category mod (ui) 💡 Feature New feature or request ❓ Feedback Further information is requested P3: Normal Priority 3
Projects
None yet
Development

No branches or pull requests

3 participants