Page MenuHomePhabricator

Prevent object titles in the "Object Attacher" dialog from triggering Quicksand "Close Dialog on Navigation" behavior
ClosedPublic

Authored by epriestley on Aug 1 2019, 7:06 PM.
Tags
None
Referenced Files
F13268546: D20693.diff
Wed, May 29, 5:09 AM
F13267866: D20693.diff
Wed, May 29, 4:20 AM
F13266040: D20693.id.diff
Tue, May 28, 8:28 AM
F13254085: D20693.diff
Sat, May 25, 3:09 AM
F13232504: D20693.diff
Tue, May 21, 1:20 AM
F13221949: D20693.diff
Sun, May 19, 3:11 AM
F13219530: D20693.id49354.diff
Sat, May 18, 7:13 PM
F13207966: D20693.diff
Thu, May 16, 6:53 AM
Subscribers
None

Details

Summary

Fixes T13363. Currently, these are genuine links which we intercept events for.

Make them pseudolinks instead. Possible alternative approaches are:

  • Keep them as genuine links, but mark them as non-navigation links for Quicksand. (But: yuck, weird special case.)
  • Keep them as genuine links, and have the dialog handler JX.Stratcom.pass() to see if anything handles the event. (But: the "pass()" pattern generally feels bad.)

"Tableaus" or whatever comes out of T10469 some day will probably break everything anyway?

Test Plan
  • Opened the "Edit Related Tasks... > Edit Subtasks" dialog.
  • Clicked task title links (not the "open in new window" icon, and not the "Select" button).
  • Before: Dialog (sometimes) closed abruptly.
  • After: Task is consistently selected as part of the attachment set.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Aug 1 2019, 7:25 PM
This revision was automatically updated to reflect the committed changes.