Page MenuHomePhabricator

When a user clicks a navigation link in a dialog, close the dialog
ClosedPublic

Authored by epriestley on Jun 6 2019, 1:50 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 10, 7:12 AM
Unknown Object (File)
Feb 7 2024, 9:38 AM
Unknown Object (File)
Feb 3 2024, 10:38 PM
Unknown Object (File)
Feb 2 2024, 8:17 PM
Unknown Object (File)
Jan 25 2024, 10:58 AM
Unknown Object (File)
Jan 25 2024, 2:05 AM
Unknown Object (File)
Jan 19 2024, 4:54 PM
Unknown Object (File)
Jan 15 2024, 6:19 PM
Subscribers
None

Details

Summary

Ref T13302. Currently, if you enable Quicksand (by clicking "Persistent Chat"), open a dialog with links in it (like "Create Subtask" with multiple available subtypes), and then follow a navigation link, the page content reloads behind the dialog but the dialog stays in the foreground.

Fix this by closing dialogs when users click navigation links inside them.

Test Plan

With Quicksand enabled and disabled, clicked a subtask type in the "Create Subtask" dialog.

  • Before, Quicksand Disabled: Dialog stays on screen, then navigation occurs.
  • After, Quicksand Disabled: Dialog vanishes, then navigation occurs.
  • Before, Quicksand Enabled: Dialog stays on screen, navigation occurs behind it.
  • After, Quicksand Enabled: Dialog vanishes, then navigation occurs.

Diff Detail

Repository
rP Phabricator
Branch
quicksand1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22987
Build 31550: Run Core Tests
Build 31549: arc lint + arc unit

Event Timeline

amckinley added inline comments.
webroot/rsrc/externals/javelin/lib/Workflow.js
98

Needs less "so", I think.

This revision is now accepted and ready to land.Jun 18 2019, 9:49 PM