Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
Seven theme
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Jan 2010 at 15:13 UTC
Updated:
4 May 2010 at 16:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
casey commentedYou mean it's not zebra style?
Comment #2
aspilicious commentedyes
Comment #3
aspilicious commentedComment #4
mr.baileysConfirmed on IE7. This is specific to Seven, the striping shows up fine when using Garland, and other 'striped' tables are also rendered without stripes in IE7 when using Seven (for example admin/config/people/permissions).
Comment #5
droplet commentedknown IE buggy ?
Comment #6
jacineHey, just came across a related issue where the links in table headers (admin/content) do not appear in IE6. This patch fixes that and the row striping issue. I put it in ie.css as it applies to both IE7 and IE6. Please review ;)
Comment #7
jacineActually, use this one. Revised the comment.
Comment #8
aspilicious commentedFixes the issue can't find any problems with the patch.
RTBC (see screenshot)
Comment #9
webchickCommitted to HEAD, with minor spelling correction in comment. Thanks!
Comment #10
jacineOh, crap. Sorry. Thank you webchick :D
Comment #11
casey commentedcrosslinking: #684626: IE BUG 3: ugly vertical stripes on some overlay pages in IE7
Comment #12
seutje commentedsubscribe
Comment #13
james.elliott commentedNew patch to fix the lack of striping and not cause the related issue in #11
- Removed the "position: relative;" from the tr which was causing all the table rows to push 1px down and right in IE7 and make the borders look broken
- Removed the "background: transparent;" from reset.css which was the root cause for why the tr.odd and tr.even backgrounds weren't displaying
Tested in FF 3.5, FF 3.6, Safari 4, Chrome 4, IE7, IE8, and Opera 10.51 and didn't notice any unexpected rendering elsewhere due to removing the "background: transparent;" from reset.css
Comment #14
james.elliott commentedSetting to needs review
Comment #15
aspilicious commentedI hink we have a winner (at least for my limited testing in IE7) It fixes a lot of IE7 bugs...
See screenshots for the ultimate magic...
Comment #16
aspilicious commentedCan we get this in, this bit me again while searching for IE7 bugs in rtl... :(
It still applies!
PLEASE! :D
Comment #17
webchickOk, since you ask so nicely... ;)
This is actually great, because it removes the previous fix that special-cased IE.
Committed to HEAD.
Comment #18
seutje commentednot to be annoying, but where did the last part of the patch come from? you don't mention it in your post and it isn't rly related to this issue... from another issue perhaps?
talking about
Comment #19
james.elliott commentedBlarg! I could have sworn I had cleaned my d7 install before creating the patch.
Those lines were for a patch to #702788: "Manage fields" tab not styled correctly on sub-pages