As derived from Texts are not vertically aligned in listing views and Dropbutton style update for Seven.
Problem/Motivation
Texts in Views list are not vertically centered within the table rows.

Expected behavior
The texts should stay vertically centered within the table rows.
Proposed resolution
Change value for vertical-align of td,th from 'top' to 'middle'.
Remaining tasks
Create a patch, test, review
User interface changes
Views list will look better
API changes
None
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | Screen Shot 2014-11-09 at 15.26.49.jpg | 374.62 KB | lewisnyman |
| #10 | Screen Shot 2014-11-09 at 15.26.41.jpg | 511.14 KB | lewisnyman |
| #9 | texts_are_not-2293875-9.patch | 511 bytes | g-raph |
| #4 | content_after_fix.png | 37.85 KB | ngocketit |
| #4 | views_after_fix.png | 71.78 KB | ngocketit |
Comments
Comment #1
ngocketit commentedPatch attached.
Comment #2
ngocketit commentedComment #3
dawehnerIt would be great to include a screenshot of after.
Comment #4
ngocketit commentedI attached some screenshots of the UI after applying the fix.
Comment #5
dawehnerIs it just me that this css selector is way to unspecific? Just curious about your thoughts here.
Comment #6
ericski commentedI would tend to agree, I'd considering using .views-table
I also question if this /should/ be the expected behavior. Personally I prefer top valign in tables, though I'm quick to admit I'm not a designer/UX expert.
Comment #7
tjtj commentedI agree with ericski. I prefer top alignment, and that is what is shown in the Views preview. But displaying the page makes it vertically centered. Why can't we choose the vertical alignment in addition to the horizontal alignment?
Comment #8
lewisnymanI'm not sure why there is any views ui specific styling for table cells. Why are they different to every other table in core? Should we be aiming for consistency instead of the preference of every module?
Comment #9
g-raph commentedI totally agree with Lewis. Since views is in core, we must be consistent and handle a global styling. There is no need for views to overwrite basic styling. If we want an addition for top alignment, then we should make a new issue from that.
I removed the overwriting styles. Patch in attach.
Comment #10
lewisnymanI can confirm that the latest patch fixes the problem and displays the Views UI listing the same as other listings, screenshots attached.
Comment #11
alexpottThis issue is a unfrozen change (css) as per https://www.drupal.org/core/beta-changes and it's benefits outweigh any disruption. Committed c93819c and pushed to 8.0.x. Thanks!