Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Hello,
I notice field name (label) is capitalized on node edit screen. Please remove capitalizatoin as I dont like "yelling" at my users. :)
I also notice the field draws a table or a border around the entire field, please remove that as well.
Cheers,
G
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | drupal-seven_css_label_fix-1299898-13.patch | 403 bytes | swarad07 |
| #7 | date-seven_css_fix-1299898-7.patch | 403 bytes | swarad07 |
| #4 | Drupal_Date_ug.JPG | 24.34 KB | warmth |
| #1 | capitalized.png | 21 KB | giorgio79 |
Comments
Comment #1
giorgio79 commentedHere is a screenshot
Comment #2
giorgio79 commentedBetter title
Also please remove the border as well, lets leave design decisions to designers :)
Comment #3
karens commentedDate module is not capitalizing anything, that is coming from your theme. The border can be overridden by anyone who knows css. Anyone who does not know css will not know how to add it and date fields are a big collection of elements that is hard to understand without some differentiation. So I'm inclined to leave the border as a default setting.
Comment #4
warmth commentedIt's also getting capitalized here and the second "Date" label is not getting translated!
Comment #5
elijah lynnI am having this issue too, it happens on a vanilla D7 too when using the default Seven theme for the admin theme. If you look at the html markup in the source code it is the correct case. So the CSS for the Seven theme is making it uppercase. This is an issue of patching Seven's CSS.
Comment #6
elijah lynnMarking as Novice.
Comment #7
swarad07This CSS fix should go in a custom theme's CSS rather than the seven theme's which is core.
anywho, It is a text-transform set to uppercase, which basically converts your label to uppercase. :-) Attaching a very small and simple patch for the same.
Some info if needed: http://www.w3schools.com/cssref/pr_text_text-transform.asp
Cheers
Comment #8
elijah lynnThanks for the patch, should we move this to Seven then?
I wouldn't think it would be good to put this on the user and their theme since everything else works except this field. The user shouldn't have to theme basic field stuff that to them appears in "core".
I guess what needs to be figured out now is what effect does setting text-transform to none have elsewhere.
I think this issue should be moved to Seven though. If not we can create a new one and cross link them.
Comment #9
elijah lynnCross linking issue I created in core for Seven theme -> http://drupal.org/node/1883544
Comment #10
webchickYeah, this is a core issue.
However, it represents a pretty significant design change, and we can't really do it in D7. Could talk about it in D8, though.
Comment #11
rteijeiro commentedIt seems to be solved in 8.x so no need to fix ;)
Comment #13
swarad07Attaching the patch again.
Comment #14
dcam commentedAs @webchick mentions in #10 we can't make this change to 7.x. If you are interested in patching your own installation, check out the patch in #13.