Comments

casey’s picture

You mean it's not zebra style?

aspilicious’s picture

yes

aspilicious’s picture

Title: IE BUG 2: no colour difference between "recent content" in dashboard » IE BUG 2: no zebra style: "recent content" in dashboard
mr.baileys’s picture

Title: IE BUG 2: no zebra style: "recent content" in dashboard » IE BUG 2: No table-striping in Seven.
Component: dashboard.module » Seven theme

Confirmed 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).

droplet’s picture

known IE buggy ?

jacine’s picture

Status: Fixed » Needs review
StatusFileSize
new560 bytes

Hey, 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 ;)

jacine’s picture

Status: Active » Needs review
StatusFileSize
new532 bytes

Actually, use this one. Revised the comment.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new244.4 KB

Fixes the issue can't find any problems with the patch.
RTBC (see screenshot)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD, with minor spelling correction in comment. Thanks!

jacine’s picture

Oh, crap. Sorry. Thank you webchick :D

casey’s picture

Status: Needs review » Needs work
seutje’s picture

subscribe

james.elliott’s picture

StatusFileSize
new1.57 KB

New 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

james.elliott’s picture

Status: Needs work » Needs review

Setting to needs review

aspilicious’s picture

StatusFileSize
new242.35 KB
new352.93 KB

I 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...

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

Can we get this in, this bit me again while searching for IE7 bugs in rtl... :(
It still applies!

PLEASE! :D

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Ok, since you ask so nicely... ;)

This is actually great, because it removes the previous fix that special-cased IE.

Committed to HEAD.

seutje’s picture

not 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

Index: themes/seven/style.css
===================================================================
RCS file: /cvs/drupal/drupal/themes/seven/style.css,v
retrieving revision 1.49
diff -u -p -r1.49 style.css
--- themes/seven/style.css	20 Mar 2010 14:45:05 -0000	1.49
+++ themes/seven/style.css	2 Apr 2010 02:42:13 -0000
@@ -293,7 +293,8 @@ ul.primary li a,
 ul.primary li a.active,
 ul.primary li a:active,
 ul.primary li a:visited,
-ul.primary li a:hover {
+ul.primary li a:hover,
+ul.primary li.active a {
   background-color: #a6a7a2;
   color: #000;
   font-weight: bold;
james.elliott’s picture

Blarg! 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

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.