Problem/Motivation
As a consequence of this Dropbutton style update for Seven, the vertical-align of td,th is changed from 'middle' to 'top' causing bad alignment of the texts inside certain listing views such as Content, People, Views etc.


Expected behavior
The texts should stay vertically centered within the table rows.
Proposed resolution
Restore 'middle' value for vertical-align of td,th and at the same time make sure the alignment of buttons is not affected.
Remaining tasks
Remove !important from latest patch, making sure the selector is still strong enough to apply.
Anti regression screenshots
User interface changes
Listing views created with tables will look better
API changes
None
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | 2293589.jpg | 1.15 MB | g-raph |
| #12 | texts_are_not-2293589-12.patch | 850 bytes | g-raph |
| #12 | interdiff.txt | 494 bytes | g-raph |
| seven_views_listing_text_alignment.png | 26.45 KB | ngocketit | |
| seven_content_listing_text_alignment.png | 30.86 KB | ngocketit |
Comments
Comment #1
ngocketit commentedComment #2
ngocketit commentedPatch attached.
Comment #3
ngocketit commentedA similar patch for Views is needed too but I guess we should do that in a new issue.
Comment #4
lauriiiI think we can do the change needed for Views also in this issue.
Comment #5
lewisnymanI actually think we should do the views patch separately, because the Views UI might require a slightly different design.
I'm not sure why we have these changes?
It looks like middle is actually the default browser styling, so we could delete this CSS completely?
Comment #6
ngocketit commentedI created a new issue and attached a simple patch for views_ui at Texts are not vertically aligned in Views list. About the default browser styling, it seems that the value is 'baseline' (vertical-align default value), not 'middle' and since the implementation may differ from browser to browser, I think it's no harm to keep 'vertical-align: middle' explicitly.
Comment #7
ngocketit commentedComment #8
g-raph commentedI agree. "Vertical-align: middle" is no harm. I reviewed this patch and it fixes this issue. But since the latest version of core, the 'core/theme/seven/style.css' file doesn't exist anymore. Now the styling is in '/core/themes/seven/components/tables.css'.
So new patch in attach. (And let's hope that sevens filestructure doesn't change anymore.)
Comment #9
nathanlawsn commentedTested the patch, all good.
Screenshot attached.
Comment #10
lewisnymanI don't see any dropbuttons in the above screenshot? We need screenshots from a variety of pages to make sure we don't cause regressions elsewhere. Here are some suggestions:
Also the suggestions I made in #5 are still present so keeping this on needs work. We shouldn't need to use !important. I looked up the documentation for vertical align and the default is actually baseline, so we don't need to change this. I updated the remaining tasks.
Comment #11
nathanlawsn commentedOops, sorry. First time getting involved in the issue queues, so please excuse me if I overlook something.
Screenshots attached of dropbuttons. The vertical alignment looks to be unaffected, though I have noticed the font size of the dropbuttons on the Views listing/edit page is inconsistent with those on the Content type listing.
Table rows are still top vertically aligned on the Views listing page also.
Comment #12
g-raph commentedI did the asked change from #10.
I removed the !important on the dropdown-widget position: static. And indeed, there is no need to use an !important.
New patch in attach!
Here are some testing screenshots, looks good IMO:

Comment #13
eftimitac commentedVerified on Chrome(Mac) and IE9(Win7)
Looks much better now, but can you please vertically inline description row for admin/structure/views page?
See the screenshot:
http://goo.gl/oEnomd
Comment #14
g-raph commentedHi eftimitac,
Views table alignment is in a new issue "Texts are not vertically aligned in Views list" (see comment #5 and #6).
Comment #15
lewisnymanI also ran over all the pages, just to double check. I think we are good to go. This only affects the Seven theme.
Comment #16
alexpottCommitted 63a6e44 and pushed to 8.0.x. Thanks!
Comment #19
bwinett commentedDocumenting that #2349829: #2293589 breaks table row buttons now lists this issue as its parent. Note that #2349829: #2293589 breaks table row buttons is now RTBC.