embed: logo icon opens destination URL inside frame

Bug #357380 reported by mangtronix
2
Affects Status Importance Assigned to Milestone
Internet Archive BookServer
New
Undecided
Unassigned

Bug Description

In the embed case the logo icon (IA logo in our case) opens the target URL inside the window. Not sure what the desired behaviour is here. We could open the target URL in a new window or reach up out of the frame and replace the location for the parent document (not sure if this is possible with current browsers).

Originally reported here: https://bugs.edge.launchpad.net/gnubook/+bug/356650

mangtronix (mang)
Changed in gnubook:
status: New → Confirmed
Revision history for this message
mangtronix (mang) wrote :

I'm setting the link target to _top for now since that will work for embed and non-embed. The correct setting for embed will be _new but we will revisit that in a more general push to make embed cleaner.

Revision history for this message
mangtronix (mang) wrote :

I set the target to _blank in the embed case, so clicking the logo will open a new window.

Changed in gnubook:
assignee: nobody → Bonnie Real (bonnie-archive)
status: Confirmed → Triaged
Revision history for this message
mangtronix (mang) wrote :

Bonnie please check that clicking on the Archive logo opens a new window when clicked in the embedded book reader. When in a full window clicking the link should navigate to archive.org.

Changed in gnubook:
importance: Undecided → Medium
milestone: none → 0.9.9
tags: added: needs-qa
Revision history for this message
Bonnie Real (bonnie-archive) wrote :

It does! The only exception is IE8, in which case archive.org does open in a full window (previously it opened right there in the embed frame), but it opens in the window you are presently in, not a new window.

tags: added: qa-verified
removed: needs-qa
Changed in gnubook:
assignee: Bonnie Real (bonnie-archive) → mangtronix (mang)
Revision history for this message
mangtronix (mang) wrote :

Hmm works for me on IE8 on XP SP3.

mangtronix (mang)
Changed in gnubook:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.