search

daem-on / wtherion

A web-based Therion map editor
https://daem-on.github.io/wtherion/
MIT License
17 stars 2 forks source link

Is it possible to grey out inactive scraps? #21

Open CaverBruce opened 2 years ago

CaverBruce commented 2 years ago

I may not have thought this through properly, however... Objects in inactive scraps are hard to distinguish from objects in active scraps. We can hide inactive scraps but often I want to see the inactive scrap objects as context for the scrap I am drawing. Inactive scrap objects can be identified by the fact that they cannot be selected, but this is not sufficient for me where the drawing might be a little complex. Is it possible to make the colour/rendering of inactive scraps recessive or greyed out by default? This way we can see them and use them for context while drawing adjacent scraps, but not inadvertently think they are part of the scrap we are currently drawing.

daem-on commented 2 years ago

I was going to do this with transparency, but due to issues in paper (the framework I'm using) or HTML canvas itself, large amounts of half-transparent objects were extremely slow. So I gave up on this idea. We could however do recolouring like XTherion does, though I find it very visually unappealing. This would mean recolouring every object every time a scrap change happens, but that should not be a problem on reasonably sized project files - especially since you can hide the other scraps and only show three or four at a time.

CaverBruce commented 2 years ago

My eyes are not great. I find the WTherion rendering towards the too fine and subtle end of the spectrum (but don't change it - maybe that is a configuration option one day). But I think transparency on rendered objects would be a step too far for me. I would prefer greyed inactive objects as implemented in XTherion. image

Shemesh commented 2 years ago

great addition! like it very much! i suggest to make the not active color a bit more "fade" meaning the grayish color bit more toward the whites.

CaverBruce commented 2 years ago

(deleted my last comment) Took me a while to find/notice that this new behaviour is controlled in the configuration panel. image I like it.

Notice the scrap panel has changed a bit - probably for the better. One oddity is that the current scrap can be made invisible.
image Is that intended behaviour?

daem-on commented 2 years ago

I didn't know that this is new behaviour, I thought you could change the visibility of any scrap, regardless of it being active (?). But yes I think this is how it should work anyway.

daem-on commented 2 years ago

Also, yes the localisation is still missing for that config option.