Ticket #2878 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch; positive review] notebook -- cython .c and .html links should open in new links (use target="_new"

Reported by: was Owned by: boothby
Priority: major Milestone: sage-3.0
Component: notebook Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description (last modified by was) (diff)

Do this, because otherwise user (1) clicks html or c link, then (2) sees the html or c, then (3) presses browser back button, then (4) PANICS! since all their work is gone (actually they can get it back by pressing refresh). This is VERY confusing, and I've had users ask about it multiple times recently.

Alternatively, is there a way so that when a user navigates to a worksheet page via the back button, the page is automatically refreshed?

Attachments

sage-2878.patch Download (955 bytes) - added by was 5 years ago.

Change History

comment:1 Changed 5 years ago by was

  • Description modified (diff)

Changed 5 years ago by was

comment:2 Changed 5 years ago by was

  • Summary changed from notebook -- cython .c and .html links should open in new links (use target="_new" to [with patch; needs review] notebook -- cython .c and .html links should open in new links (use target="_new"

comment:3 Changed 5 years ago by boothby

  • Summary changed from [with patch; needs review] notebook -- cython .c and .html links should open in new links (use target="_new" to [with patch; positive review] notebook -- cython .c and .html links should open in new links (use target="_new"

"Alternatively, is there a way so that when a user navigates to a worksheet page via the back button, the page is automatically refreshed?"

This is a hard problem. Let's think about making a new ticket out of that issue (because the problem is ultimately larger than these links), and take this patch for now.

comment:4 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged in Sage 3.0.alpha4

Note: See TracTickets for help on using tickets.