This puts the preview where it's most needed and visible; to the right of the palette.

Coupled with #828578: Make clicks on theme settings preview select palette fields to edit would help out the UX of using color.module in Seven.

Attaching patch and screenshots comparing before-after while configuring Garland.

Issue fork drupal-831736

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

janusman’s picture

Forgot to mention this requires clearing caches as it overrides a default theming function.

aspilicious’s picture

Will not work for small resolutions

janusman’s picture

Hmm, maybe I can add in some JS can add in a class to the #edit-color fieldset (that would enable the new CSS rules) only if it is more than a certain minimum width upon loading. Of course, then it would also need to handle resizing...

aspilicious’s picture

Yes and that will lead to unnecessary code. For me this is a won't fix.

slawira’s picture

Issue summary: View changes
Status: Needs review » Needs work

Patch no longer apply and also there are white space issues. I think we should use the media query for smaller resolutions @aspilicious.

LewisNyman’s picture

Issue tags: +frontend, +CSS

I don't mind this design improvement but it seems really complicated. We have layout classes insystem.admin.css that you can use to achieve this without writing any CSS and JS.

aspilicious’s picture

5 years ago I wasn't aware of that @slawira ;)

akashdab made their first commit to this issue’s fork.

akashdab’s picture

Status: Needs work » Needs review

Positioned color.module preview to suit larger screens better