angular: modals do not automatically close when navigating away from pages

Bug #1849372 reported by Andrea Neiman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.3
Fix Released
Medium
Unassigned
3.4
Fix Released
Medium
Unassigned

Bug Description

3.4 (probably all new Angular interfaces)

If you have a modal open in a new Angular interface and click the back/forward browser buttons, the modal persists over the (unrelated) pages until you hit an AngularJS interface.

Bill Erickson (berick)
Changed in evergreen:
status: New → Confirmed
assignee: nobody → Bill Erickson (berick)
Revision history for this message
Bill Erickson (berick) wrote :

Fix pushed:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1849372-ang-route-close-dialog

To test:

[1] Navigate to Server Administration
[2] Navigate to Age Hold Protection Rule Configuration
[3] Double-click a grid row to open an edit dialog
[4] Click browser back button to return to the Server Admin page
[5] Confirm edit dialog closes once the navigation is complete.

tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.3.5
assignee: Bill Erickson (berick) → nobody
status: Confirmed → New
Galen Charlton (gmc)
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Galen Charlton (gmc) wrote :

Tested and signed off: user/gmcharlt/lp1849372_signoff

I note that there is a possible enhancement: rather than unconditionally dismissing the modal, it could be handle to allow an open modal prevent or override navigation if it has a dirty state. In that case, the page that is displaying the modal should display a toast or some other way of telling the user why navigation is being blocked.

tags: added: signedoff
Changed in evergreen:
assignee: Galen Charlton (gmc) → nobody
Revision history for this message
Mike Rylander (mrylander) wrote :

I've picked this into master, 3.4, and because it came in cleanly, into 3.3. 3.2 was not a clean merge, so I've left it out.

Thanks, Bill and Galen!

Changed in evergreen:
status: Confirmed → Fix Committed
Remington Steed (rjs7)
Changed in evergreen:
milestone: 3.3.5 → 3.5-alpha
Changed in evergreen:
status: Fix Committed → Fix Released
status: Fix Released → Fix Committed
Changed in evergreen:
status: Fix Committed → 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.