The description uses way to much space. And on mobile it's even worse :)


| Comment | File | Size | Author |
|---|---|---|---|
| #10 | wider_display.png | 399.92 KB | andybroomfield |
| #10 | narrower_display.png | 369.42 KB | andybroomfield |
| #10 | Screen Shot 2015-01-17 at 17.06.09.png | 434.73 KB | andybroomfield |
| #9 | description-display-settings-2372023-screenshot9.png | 112.95 KB | tompagabor |
| #9 | description-display-settings-2372023-9.patch | 1.88 KB | tompagabor |
Comments
Comment #1
lewisnymanIt looks like this line of CSS prevents the text from wrapping. I don't understand why this line exists, here's a patch to remove, we should test to see if this breaks anything.
Comment #2
BarisW commentedPatch applies nicely and solves the problem. Looks good to me.
Comment #3
ericmulder1980 commentedPatch applies nicely and after a cache clear results are looking good.
Tested in Chrome 38, Firefox 33.1 and Safari 7.0.6.
Comment #4
ericmulder1980 commentedComment #5
alexpottLooks like this could remove a lot of CSS...
... and maybe more.
It'd be good to find out why that is there.
This was added to misc/drupal.css by the following commit
So no idea of the issue that prompted the commit :)
Comment #6
BarisW commentedWow, a 2004 commit? I'll try to remove the unneeded CSS too.
Comment #7
BarisW commentedI removed it where possible. On some places, the
white-space: nowrapwas added on purpose (eg in#system-modules details), so I left these places untouched.The /admin/reports/translations looks damaged anyhow, even without this patch, but I'll create a new issue for that.
Comment #8
Vally79 commentedTried to apply the last patch and it worked until the last 3 lines, which it didn't find in the file. I added those lines manually and verified that the rest of the patch applied successfully. Unfortunately, I don't see any change on Chromium 39.0.2171.65 and Firefox 33.0 on Ubuntu 14.04 LTS.

Comment #9
tompagabor commentedReroll patch, just remove the first change. Also tested it, and working goood.
Comment #10
andybroomfieldI tested the patch and it fixes the issue (The description text is wrapping now).
Comment #11
webchickOh, very nice. We fix a bug *and* delete code at the same time!
Committed and pushed to 8.0.x. Thanks!