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

Comments

carlos8f’s picture

carlos8f commented
Status: Active » Needs review
StatusFileSize
new 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 commented
Issue tags: +overlay

adding tag

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
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 🇨🇦
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.