Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

[ui] Capitalization #42

Open
virtualritz opened this issue Aug 17, 2022 · 1 comment
Open

[ui] Capitalization #42

virtualritz opened this issue Aug 17, 2022 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@virtualritz
Copy link
Contributor

Menu capitalization doesn't adhere to platform guidelines for Windows/macOS/Linux.

View graph source should be View Graph Source.

Node name capitalization is all over the place. Some nodes names use headline capitalization, others use sentence capitalization.

Labels on nodes seem to consistenly use lowercase but toggles/buttons (Active/Set active) then again use sentence capitalization.

I would open a PR with fixes but first it needs to be clarified what the rules to be enforced are. :)

I can make a suggestion if you didn't think about this yet -- HCI/UI/UX is pet peeve. :P

@setzer22
Copy link
Owner

setzer22 commented Aug 17, 2022

Hi! I don't have a lot of experience in this area, but any changes that improve UI consistency and adhere to well-known guidelines are welcome :)

Labels on nodes seem to consistenly use lowercase

The reason for that is that these labels are tied to the parameters defined in the Lua code, and these are required to be valid identifiers. I've been meaning to add a distinction between the internal parameter name and the user readable name. That would allow using consistent capitalization everywhere 👍

@setzer22 setzer22 added enhancement New feature or request help wanted Extra attention is needed labels Sep 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants