The javascript that transforms /admin path links into overlay paths (i.e., with #overlay=admin/path...) is broken in IE7, IE8, and Opera 9 and 10. I'm currently debugging this issue.

CommentFileSizeAuthor
#1 overlay-isadminlink.patch1012 bytescarlos8f
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

carlos8f’s picture

carlos8f CreditAttribution: carlos8f commented
Status: Active » Needs review
FileSize
overlay-isadminlink.patch1012 bytes

I found the cause to be that IE and Opera's implementation of the Link.pathname property lacks a leading slash. Here's a simple fix, adding the leading slash if it is absent.

eigentor’s picture

eigentor CreditAttribution: eigentor commented
Issue tags: +overlay

adding tag

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Reviewed & tested by the community

I've verified the issue on Opers 10.10 / Mac and it does indeed happen to be there. The provided patch fixes this issue so that the admin paths are properly identified. Opera 10.10 still does not open any overlays, but it does not go to different admin pages either, since fragments are now properly used and updated (so we are one step closer to Opera support but not quite there yet).

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Well that definitely falls under the category of un-good.

Committed to HEAD. Thanks a lot, Carlos!

Status: Fixed » Closed (fixed)
Issue tags: -overlay

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

Issue tags: +overlay

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.