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.
Hi,
See the attached image, when secondary tabs are expanded on hover they are rendered behind other elements.
A trivial patch is on its way.
Thanks,
Antonio
Comment | File | Size | Author |
---|---|---|---|
#3 | Screen shot 2011-09-09 at 10.17.19 AM.png | 30.14 KB | ericduran |
#1 | 0001-Issue-1229912-by-ao2-render-secondary-tabs-on-top-of.patch | 961 bytes | ao2 |
#1 | 02-rubik-secondary-tabs-z-index-after.png | 21.78 KB | ao2 |
01-rubik-secondary-tabs-z-index-before.png | 19.35 KB | ao2 |
Comments
Comment #1
ao2 CreditAttribution: ao2 commentedPatch attached, it can be applied with
git am
.And an image to show the result.
Thanks,
Antonio
Comment #2
derjochenmeyer CreditAttribution: derjochenmeyer commentedPatch in #1 works. One line of CSS.
Comment #3
ericduran CreditAttribution: ericduran commentedI try to replicate this, but I can't seem to get the secondary tab to render behind any elements. How can I replicate it?
I added some new links just to test. This is what I see.
I know is just a single line of css, but it'll be nice to make sure we need it before adding it.
Comment #4
derjochenmeyer CreditAttribution: derjochenmeyer commentedIt only happens on pages with .fieldset.titled elements like admin/content/node
I ran into this bug reviewing "newsletter module" (http://drupal.org/project/newsletter). To reproduce install the newsletter module and visit admin/newsletters/templates.
Comment #5
fxarte CreditAttribution: fxarte commentedThe patch works
Comment #6
melon CreditAttribution: melon commentedThe patch is working for me as well, thank you Antonio for pointing this out.
When is this expected to be committed?
Thanks
Comment #7
mpotter CreditAttribution: mpotter commentedPatch works for me also and is critical when dealing with Media module style configuration. This *really* needs to get committed.
Comment #8
ao2 CreditAttribution: ao2 commentedTo project maintainers, you know, you can use
git am
to merge the change, this way the commit message and the authorship in the patch I generated withgit format-patch
are preserved.Ciao,
Antonio
Comment #9
CoffeyMachine CreditAttribution: CoffeyMachine commentedThis issue must have been fixed by another patch, because I cannot reproduce it on the dev branch of Rubik. I installed both the Media module and the Newsletter module and was unable to reproduce the issue on the pages mentioned above.
I'm also pretty confused by the content of the patch as it adds a z-index to the div.pager rule, which is for pagers. The screenshots show the drop down menus that are hit by the .secondary-tabs rule. .secondary-tabs needed the z-index to fix the issue, which it has now on the dev branch.
Regardless, this doesn't seem to be an issue anymore so I am changing the status to fixed. If I missed something please reopen it.
Comment #10
ao2 CreditAttribution: ao2 commented@CoffeyMachine the patch I attached does add the
z-index
to.secondary-tabs
, it is just the diff header which does not reflect this. I thinkdiff -p
was meant for C-like code, it can get confused with other languages.I cannot see the change which adds the
z-index
to.secondaty-tabs
, can you paste a link to the commit?I haven't tested yet if the patch is still needed, I'll let you know.
Comment #12
TripleEmcoder CreditAttribution: TripleEmcoder commentedPatch works, and is still needed in the recent -dev.
Comment #13
shiroitatsu CreditAttribution: shiroitatsu commentedStill not committed in beta7. Took me quite a while to see that reason for some links not appearing on secondary menu was cause by this little bugger as it very neatly hides the third+ menu items under the box below. Webform and Panels also suffer from this.
Comment #14
CoffeyMachine CreditAttribution: CoffeyMachine commentedAh ok, I misunderstood the patch code. I'm still having trouble reproducing this issue, but I have gone ahead and committed the patch anyway.
Thank you!
Comment #16
ao2 CreditAttribution: ao2 commented@CoffeyMachine, thanks for applying the patch, I can see it in commit 2289259.
Next time, please, consider using
git am
to apply my patches (which I prepare carefully and export withgit format-patch
), in doing so I will figure as the author in the git history (that's stronger than the "by ao2" in the short commit message), my patch also had a long commit message explaining the reason WHY the patch was needed, that has been stripped out as well.Loosing attribution and see the work on composing an useful commit message wasted could discourage further contributions.
And when an issue is closed, I think that adding a link to the commitdiff would also improve the communication.
Just my 2c, thanks for your work on Rubik.
Regards,
Antonio