search

rbelow / blueprintcss

Automatically exported from code.google.com/p/blueprintcss
1 stars 0 forks source link

td/th styles bust vertical rhythm #80

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
The pixel padding on the table cell styles bust the vertical rhythm.

Original issue reported on code.google.com by fla...@gmail.com on 17 Feb 2008 at 12:27

GoogleCodeExporter commented 8 years ago 
Known problem. As the line height is 18px, this is hard to fix.

If each table cell were to follow the rhythm, they'd have to have a height of 
18 or 36 pixels. The first one is too 
small, the second too large. 

Suggestions?

Original comment by ola...@gmail.com on 18 Feb 2008 at 7:56

GoogleCodeExporter commented 8 years ago 
My first choice would be to use 18 pixels, even if it is small.

Otherwise...maybe use a height of 27 pixels and require an even number of rows 
to
maintain rhythm?

Original comment by fla...@gmail.com on 19 Feb 2008 at 8:51

GoogleCodeExporter commented 8 years ago 
Interesting idea. Thanks, I'll have a go at it. :)

Original comment by ola...@gmail.com on 20 Feb 2008 at 8:30

GoogleCodeExporter commented 8 years ago 
Report copied to
http://blueprintcss.lighthouseapp.com/projects/15318-blueprint-css/tickets/2-td-
th-styles-bust-vertical-rhythm

Original comment by horst.gu...@gmail.com on 12 Aug 2008 at 7:30