Following a persistent idea from admin themes such as root candy, I've written this patch to use color codes in buttons.
It will create three standard classes to be used in forms

- button-main
- button-warning
- link-warning

The first in intended to be used with the main action button in form (Save settings, Publish node, Apply)
The other two are intended to be used with potentially destructive actions, such as Delete, either with buttons or links.
This patch changes all core forms to use those three classes and implements the color codes in Garland

Notes:
The Garland/Minnelli base.png images are in the .tar.gz file.
I made a screenshot with the Garland default and Shiny Tomato colors schemes.

CommentFileSizeAuthor
screenshot.png4.26 KBdanillonunes
buttons_styles-25-05-09.tar_.gz62.46 KBdanillonunes
buttons_styles-25-05-09.patch36.11 KBdanillonunes

Comments

danillonunes’s picture

danillonunes commented
Issue tags: +Usability, +D7UX

adding tags

ejort’s picture

ejort commented
Status: Active » Needs work

I like this idea and it's definitely something that I think is missing from Drupal's forms (giving a heirarchy to the buttons). I had a quick scan of the patch, and it is mostly fairly straight forward. One thing that I would question however is:

+  -moz-border-radius: 3px;
+  -moz-outline-radius: 3px;
+  -webkit-border-radius: 3px;
+  -webkit-outline-radius: 3px;
+}

You should add the standard version of 'border-radius' at the end of the vendor specific declarations so that current / future browsers that support the standards based property will pick this up as well (and in preference to the vendor specific versions). 'outline-radius' isn't a part of the CSS standards as far as I know so this doesn't apply to it.

eg:

+  -moz-border-radius: 3px;
+  -moz-outline-radius: 3px;
+  -webkit-border-radius: 3px;
+  -webkit-outline-radius: 3px;
+  border-radius: 3px;
+}
Noyz’s picture

Noyz commented

Not a fan personally. Buttons and links need to be really consistent. If there are warnings on the page, the warnings should be carried through via messaging. Messaging is more appropriate to color coordinate. I could see this getting out our hand where messages and buttons are both color coordinated - thereby creating a kaleidoscope of a UI.

Bojhan’s picture

Bojhan commented
Status: Needs work » Closed (won't fix)

Sorry, this is indeed not consistent and it would really distract from one filling out a form.