Skip to content

Conversation

ekzyis
Copy link
Member

@ekzyis ekzyis commented Aug 27, 2025

Description

This PR replaces the text 'bot' if an item was created with an API key with an icon like we do for anon, hat, gun and horse.

If we can find appropriate icons, we could also do the same for 'outlawed' and 'freebie', but they are not part of this PR.

Additionally to less visual clutter, this would also make the territory name stand out more.

Screenshots

before:

2025-08-27-134215_551x52_scrot

after:

2025-08-27-134237_538x48_scrot

example image with placeholder for freebie icon:

2025-08-27-134717_499x51_scrot

Checklist

Are your changes backward compatible? Please answer below:

yes

On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:

9. item info for posts, comments renders fine

For frontend changes: Tested on mobile, light and dark mode? Please answer below:

yes, also looks good in light mode

Did you introduce any new environment variables? If so, call them out explicitly here:

no

@ekzyis ekzyis added enhancement improvements to existing features ui/ux labels Aug 27, 2025
@huumn
Copy link
Member

huumn commented Aug 28, 2025

tbh this doesn't seem less cluttered to me in an obvious way. it's kind of trading one form of clutter for another that's less explicit

@ekzyis
Copy link
Member Author

ekzyis commented Aug 28, 2025

Mhh, maybe this was a premature solution to my perceived problem that the territory name doesn't stand out enough for how important it should be. It's currently shown equally as important as if an item was created by a bot, for free, was outlawed or is the top boost.

@huumn
Copy link
Member

huumn commented Aug 29, 2025

Ah I get the motivation now.

I'd merge this if we don't display hat/horse/gun on bot posts/comments and just this icon instead. Otherwise, I'm not sure it's worth it for either motivation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement improvements to existing features ui/ux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants