Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
See http://img.skitch.com/20100222-bce3ncm71apxpshtsse145q5u.jpg - it's an odd bug, but it's pretty consistent on my machine.
Comment | File | Size | Author |
---|---|---|---|
#5 | chomebug.4.patch | 700 bytes | cosmicdreams |
Comments
Comment #1
cosmicdreams CreditAttribution: cosmicdreams commented@dmitrig01: Those tabs look unfamiliar to me, What page were they on? I'd like to test that myself.
Comment #2
cosmicdreams CreditAttribution: cosmicdreams commentedNever mind @dmitrig01, I was able to reproduce. Looks like the culprit is
span.element-invisible and the way it is positioned absolutely.
Researching...
Comment #3
cosmicdreams CreditAttribution: cosmicdreams commentedCan anyone comment on why this element is needed?
Comment #4
cosmicdreams CreditAttribution: cosmicdreams commentedFound the fix! Modify a.active to include white-space: pre;
Wonder if that breaks anything. If no one beats me to it. I'll make a patch for this tonight.
Comment #5
cosmicdreams CreditAttribution: cosmicdreams commentedHere's a patch that adds this one-liner. I am in the process of testing this for other browsers.
Comment #6
cosmicdreams CreditAttribution: cosmicdreams commentedSo 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,
Comment #7
cosmicdreams CreditAttribution: cosmicdreams commentedIdeally, 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.
Comment #8
sunDon'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
Comment #9
cosmicdreams CreditAttribution: cosmicdreams commentedThank 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.
Comment #10
sunSo #745542 has been marked as duplicate of a unknown issue. What's the status of this one?
Comment #11
cosmicdreams CreditAttribution: cosmicdreams commentedsorry 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.