Skip to content

Conversation

@emptybraces
Copy link
Contributor

@emptybraces emptybraces commented Apr 15, 2025

  • Add color highlighting for Tags and Layers
  • Hide label "Untagged"
  • Show full name as tooltip on mouse hover
  • Show checkmark to current on selection menu
out.mp4

- Hide tag "Untagged"
- Show full name as tooltip on mouse hover
- Show checkmark to current on selection menu
@emptybraces
Copy link
Contributor Author

There's no real value in displaying "Untagged." Colors are selected using a hash to avoid collisions, which helps distinguish labels with long names that only differ slightly at the end. Thanks to this, things have become much easier to read✌️
image

Copy link
Owner

@WooshiiDev WooshiiDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we fix the indentation on these? The class and contents have been shifted right

@emptybraces
Copy link
Contributor Author

Sorry I've fixed the indentation.🙏

@emptybraces emptybraces requested a review from WooshiiDev May 6, 2025 06:40
@WooshiiDev
Copy link
Owner

I love the use of a hash to avoid colour conflicts, but I feel the S & V is a little unintuiative for general users. I have to also ask about performance due to the use of mod, char iteration and the cast going on just for colour output.

I do love the idea of not having to assign colours per tag & layers so this is not a denial from myself 🙏 I'm wondering if there's a better way to display this information.

I also agree with untagged being wasteful to display when it doesn't do much - could this be a toggle too? My arguement for it being a toggle more so comes from those that may want it to display for any reason they require it, but also the fact it displays the selection area for changing the tag in the first place.

@emptybraces
Copy link
Contributor Author

Alright let's make 'Untagged' a toggle option😃
And will check the color options and their handling when I have time.

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