Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Updated: Comment #0
Problem/Motivation
No spacing between field label and value when set to inline display.
Before:
After:
Proposed resolution
Fix it
Remaining tasks
Review the css
User interface changes
None
API changes
None
Related Issues
None
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 2110787-5.patch | 518 bytes | swentel |
| #3 | 2110787-3.patch | 520 bytes | swentel |
| field-ui-label.patch | 514 bytes | larowlan | |
| Screenshot 2013-10-13 17.24.53.png | 31.41 KB | larowlan | |
| Screenshot 2013-10-13 17.22.09.png | 38.03 KB | larowlan | |
Comments
Comment #1
swentel CreditAttribution: swentel commentedMakes sense.
Wondering whether we should also look at the tags label and definitely the language label, because that one is really akward - but that might be fixable with #1471706: Allow hook_field_extra_fields() label to be displayed Above/Inline/Hidden, the same way fields and groups are
Comment #2
joelpittetDamn, must have cross-posted somehwere else when I replied a few days ago. I was thinking this should be 1em instead of 5px, just to keep things relative as bases change.
Comment #3
swentel CreditAttribution: swentel commented1em was a lot, so went for the straight conversion of px to em
Comment #4
joelpittetIf 1em length is too far, which is more than 5px... then 0.5em would fit the bill pretty close (6px @12px base from Bartik) Which is what most spaces are anyways.
En = 1/2em for those who are curious. *star rainbow*
http://en.wikipedia.org/wiki/En_%28typography%29
Comment #5
swentel CreditAttribution: swentel commentedHmm yeah, still looks good then.
Comment #6
joelpittetGreat, thanks @swentel and @larowlan :)
Comment #7
alexpottCommitted bc7a91a and pushed to 8.x. Thanks!