passivestar / godot-minimal-theme

Godot Minimal Editor Theme
MIT License
1.63k stars 27 forks source link

Feedback - Some suggestions #8

Open gperdomor opened 2 months ago

gperdomor commented 2 months ago

Hi @passivestar...

Thank you for this awesome theme, it's great, but I think can be improved a little bit because in some parts of the UI it's too flat, or in this case, too minimal, I don't known if it's intended 😂

Here are 2 screenshots with the default theme and other 2 with the minimal theme for comparison... Basically in the FileSystem tree, the arrow paths are missing, also some "containers" have no borders and can be a little confusing because everything looks like a label and not a clickable resource (Like the Sprite 2D in the create new node modal, below Recents label)

I hope you can consider this feedback 😄

BTW, the screenshots are for Godot 4.3 beta 2 and minimal theme high ppi v1.5

Screenshot 2024-07-03 at 8 00 44 PM Screenshot 2024-07-03 at 8 00 46 PM Screenshot 2024-07-03 at 8 03 26 PM Screenshot 2024-07-03 at 8 03 31 PM
passivestar commented 2 months ago

I don't known if it's intended 😂

It's not, it's just what we have to deal with considering godot's current theming limitations

image

Godot theming still lacks a lot, I'd very much like to have clear visible separation for those containers but since they affect all docks I had to make a choice. Fixing this will require a big amount of work on the engine side, probably something to explore in 4.4

the arrow paths are missing

some people mentioned that, I might bring those back but I'd like to see what it'll look like with just vertical lines, like it's done in vscode. might be a good compromise on minimalism/function

Thanks for the feedback! <3

gperdomor commented 2 months ago

:o... I understand... thank you for the quick response 👍🏻

Horops commented 1 month ago

Yeah I came here to say this also. The lack of line (arrow?) paths in the Scene and FileSystem windows makes it really hard to visualize what goes where.
I attached a screenshot for example (though not the best example since the tree is really small in this case so not that hard to follow but in a bigger project it gets unwieldy fast) I understand it's currently a Godot limitation though as PassiveStar said. Hope the org can make this change possible sometime soon. Untitled