After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 340742 - No history copied over when opening link in new tab/win using context menu
No history copied over when opening link in new tab/win using context menu
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: History
2.14.x
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Marco Pesenti Gritti
Depends on:
Blocks:
 
 
Reported: 2006-05-05 14:03 UTC by René Stadler
Modified: 2006-06-08 19:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Simple fix attempt (400 bytes, patch)
2006-05-05 14:09 UTC, René Stadler
accepted-commit_now Details | Review

Description René Stadler 2006-05-05 14:03:50 UTC
In a tab or window with history (where you can "go back"), open a link in a new tab by selecting "Open in new tab" from the context menu of a link.  The resulting new tab has no history associated (cannot "go back").  The same happpens when opening in a new window.

On the contrary, opening a link with a middle mouse click or Ctrl+left-click results in the new tab having a history.

So the functionality is already there, a simple patch follows that seems to fix this little inconsistency.
Comment 1 René Stadler 2006-05-05 14:09:09 UTC
Created attachment 64860 [details] [review]
Simple fix attempt

Just copied over the call to ephy_embed_shistory_copy () from ephy-tab.c:open_link_in_new () to ephy-shell.c:ephy_shell_new_tab_full ().  Seems to fix the issue (for both new tabs and windows).

Applies cleanly to 2.14.1 and CVS HEAD.
Comment 2 Christian Persch 2006-05-11 13:08:31 UTC
Thanks for the patch!

Phil: was there a reason you didn't add it in ephy-shell originally when you added the session history copying? (I can't remember.)
Comment 3 Philip Langdale 2006-05-11 14:50:47 UTC
No, I probably just forgot to handle this case. Patch looks fine.
Comment 4 Wouter Bolsterlee (uws) 2006-06-08 10:13:24 UTC
Fixed in HEAD.

2006-06-08  Wouter Bolsterlee  <uws+gnome@xs4all.nl>

    * src/ephy-shell.c: (ephy_shell_new_tab_full): copy
    history over when opening link in new tab/win using
    context menu. Fixes bug #340742, patch by René Stadler.


Chpe, can you apply this to gnome-2-14 if you decide it should go there too?
Comment 5 Christian Persch 2006-06-08 19:30:27 UTC
Ok to commit to gnome-2-14 .