See http://img.skitch.com/20100222-bce3ncm71apxpshtsse145q5u.jpg - it's an odd bug, but it's pretty consistent on my machine.

CommentFileSizeAuthor
#5 chomebug.4.patch700 bytescosmicdreams
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cosmicdreams’s picture

@dmitrig01: Those tabs look unfamiliar to me, What page were they on? I'd like to test that myself.

cosmicdreams’s picture

Never mind @dmitrig01, I was able to reproduce. Looks like the culprit is

span.element-invisible and the way it is positioned absolutely.

Researching...

cosmicdreams’s picture

Can anyone comment on why this element is needed?

cosmicdreams’s picture

Found the fix! Modify a.active to include white-space: pre;

ul.primary li.active a, ul.primary li.active a.active, ul.primary li.active a:active, ul.primary li.active a:visited {
  background-color:#FFFFFF;
  border-color:#C9CAC4;
  white-space:pre;
}

Wonder if that breaks anything. If no one beats me to it. I'll make a patch for this tonight.

cosmicdreams’s picture

Status: Active » Needs review
FileSize
700 bytes

Here's a patch that adds this one-liner. I am in the process of testing this for other browsers.

cosmicdreams’s picture

So far it looks like this solves the issue in Firefox 3.5, Chrome 5 (beta)

It didn't break IE7

It didn't solve the issue in Opera 10,

cosmicdreams’s picture

Status: Needs review » Reviewed & tested by the community

Ideally, It would be good to have another person test this patch and see that it is working fine. But I've tested this in a lot of browsers and am comfortable with the improvement it makes. I'm going to escalate this patch so that it can be better seen by committers / reviewers in hopes that it will be committed.

sun’s picture

Status: Reviewed & tested by the community » Needs review

Don't mark your own patches RTBC, please.

We should compare this solution attempt (looks a bit weird) to #745542: Wrong margin for primary active tab in Seven

cosmicdreams’s picture

Thank you sun for looking at this issue. I saw your comments on #745542: Wrong margin for primary active tab in Seven. We can mark this issue a duplicate if the discussion is flowing better there.

sun’s picture

So #745542 has been marked as duplicate of a unknown issue. What's the status of this one?

cosmicdreams’s picture

Status: Needs review » Closed (duplicate)

sorry I meant to mark this issue as the duplicate of #745542: Wrong margin for primary active tab in Seven. Sorry for stepping on the toes of the working going on over there.