I think this page needs more work.
There are quite a few differences between what this page looks like when Bartik is the admin theme vs. when Seven is the admin theme. See the attached screenshot for a comparison.
I'm also really not a fan of the #preview h1, etc code this preview is having us add. I would prefer to see it all in the preview.css file. I understand it's like that for people that may end up changing these things, but I don't think the preview needs to be flexible in this way at all. For example, I think using pixels sizes for fonts here is totally fine, given the special circumstances of having to deal with Seven's massive reset and other CSS files. It's also likely going to be a problem with other admin themes people decide to install down the line.
|#54||color-palette-css.patch||2.42 KB||Jeff Burnz|
|PASSED: [[SimpleTest]]: [MySQL] 23,297 pass(es). |
[ View ]
|#49||bartik-color-settings-2.png||37.3 KB||Jeff Burnz|
|#48||bartik-color-settings.png||16.44 KB||Jeff Burnz|
|#48||bartik-color-settings-783574.patch||1.71 KB||Jeff Burnz|
|PASSED: [[SimpleTest]]: [MySQL] 22,997 pass(es). |
[ View ]
|#47||bartik-color-settings-font-resize.png||44.1 KB||Jeff Burnz|