Paths

Table of Contentst

Differential D20693

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

Authored by epriestley on Aug 1 2019, 7:06 PM.

Tags

None

Referenced Files

	F18812130: D20693.id49353.diff
	Mon, Oct 20, 7:11 AM

	F18795031: D20693.diff
	Fri, Oct 17, 1:26 AM

	F18784090: D20693.id49354.diff
	Mon, Oct 13, 11:51 AM

	F18756128: D20693.id.diff
	Oct 5 2025, 9:23 AM

	F18696397: D20693.diff
	Sep 27 2025, 7:29 AM

	F18687527: D20693.diff
	Sep 26 2025, 2:36 PM

	F18624073: D20693.id.diff
	Sep 15 2025, 6:36 PM

	F18601536: D20693.diff
	Sep 13 2025, 12:46 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13363: In "Edit Subtasks" dialog, clicking a task tile sometimes (?) closes the dialog
Commits: rP3069ef41662d: Prevent object titles in the "Object Attacher" dialog from triggering Quicksand…

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

Branch

sub1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 23218
Build 31892: Run Core Tests
Build 31891: arc lint + arc unit

Event Timeline

epriestley created this revision.Aug 1 2019, 7:06 PM

Harbormaster completed remote builds in B23218: Diff 49353.Aug 1 2019, 7:08 PM

epriestley requested review of this revision.Aug 1 2019, 7:08 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 1 2019, 7:25 PM

Closed by commit rP3069ef41662d: Prevent object titles in the "Object Attacher" dialog from triggering Quicksand… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP3069ef41662d: Prevent object titles in the "Object Attacher" dialog from triggering Quicksand….

Revision Contents
Changeset List

Path

Size

resources/

celerity/

webroot/

rsrc/

js/

core/

behavior-object-selector.js

2 lines

Diff 49353

resources/celerity/map.php

Loading...

webroot/rsrc/js/core/behavior-object-selector.js

Loading...