wpaccessibility / settings-api-enhanced

An improved WordPress Settings API with default render callbacks and a new accessible layout.
9 stars 0 forks source link

CSS styling suggestions #42

Open hedgefield opened 7 years ago

hedgefield commented 7 years ago

I'm really digging the new way to present the information on the Settings pages, but visually I got a bit of a cluttered, disjointed feeling from glancing over the page. It's hard to describe, but it felt like it was hard to grasp what belonged together, a sort of mental noise. So I tweaked some CSS to see if I could get rid of that feeling. Tell me what you think:

general_settings_ _wordpress_develop_ _wordpress

joyously commented 7 years ago

I got that feeling too, and wouldn't know what to call it. This looks better to me.

felixarntz commented 7 years ago

The layout appears more clear to me as well with your changes - great! I like the idea of moving descriptions above the inputs in particular. While it is a bit different from what we're used to, I think it makes sense for the reading flow.

More feedback from an actual designer would be much appreciated.

afercia commented 7 years ago

Definitely something worth experimenting :) I'd just suggest to consider a max-length for long lines of text. Ideally, for better readability they shouldn't exceed ~80 characters. See also https://core.trac.wordpress.org/ticket/34991