search

KayelGee / token-attacher

MIT License
11 stars 18 forks source link

Quick Edit toggle button to use Foundry styling #79

Closed LukasPrism closed 1 year ago

LukasPrism commented 2 years ago

Just a trivial thing really, although it may help user interaction. The Quick Edit toggle button in the toolbar should probably be a regular toggle style button (ie. purple outline with a highlight when activated).

KayelGee commented 2 years ago

What do you mean by purple? Default foundry is using orange as a highlight. I didn't bother with adding an actual toggle to the token controls ui, because it's relevant for different layers(that's why I chose to show the big notifier at the bottom of the screen). But I guess to make it consistent it should be an actual toggle. Should be simple enough to add.

LukasPrism commented 2 years ago

Hmm maybe it's something I have installed, but I've seen the same on most screenshots of foundry on the discord. Orange for currently selected tool, purple for off/on toggles.

image

The classes are

controls ol.control-tools > li.toggle

controls ol.control-tools > li.toggle.active

controls ol.control-tools > li.toggle:hover

KayelGee commented 2 years ago

I checked again and yeah you're right the toggle buttons are actually purple in default foundry my bad.