When a user cancels a new inline, clear it from the objective list
ClosedPublic
Actions

Authored by epriestley on May 20 2017, 12:35 PM.

Tags

None

Referenced Files

	F19582514: D17979.diff
	Sun, Feb 1, 10:34 PM

	F19154217: D17979.diff
	Dec 11 2025, 1:53 PM

	F19099358: D17979.id.diff
	Dec 4 2025, 7:47 PM

	F19091038: D17979.diff
	Dec 3 2025, 1:22 PM

	F19016237: D17979.id.diff
	Nov 23 2025, 3:12 AM

	F18803199: D17979.id43249.diff
	Oct 17 2025, 10:03 PM

	F18753609: D17979.id.diff
	Oct 4 2025, 7:35 PM

	F18741494: D17979.diff
	Oct 2 2025, 1:28 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T12733: (2017 Week 20) Inline Comments Errata / Feedback

Commits

rPaabb791c174b: (stable) When a user cancels a new inline, clear it from the objective list
rP7a40dd380ee7: When a user cancels a new inline, clear it from the objective list

Summary

Ref T12733. Currently, creating a new inline and then canceling it leaves a marker in the objective list. Instead, remove the marker.

Test Plan

Created an empty inline, cancelled. Created a non-empty inline, cancelled. No objective marker in either case.
Created a new normal inline, objective marker.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 20 2017, 12:35 PM

Harbormaster completed remote builds in B17144: Diff 43236.May 20 2017, 12:36 PM

chad accepted this revision.May 20 2017, 2:24 PM

This revision is now accepted and ready to land.May 20 2017, 2:24 PM

Closed by commit rP7a40dd380ee7: When a user cancels a new inline, clear it from the objective list (authored by epriestley, committed by epriestley). · Explain WhyMay 20 2017, 2:59 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

12 lines

webroot/

rsrc/

js/

application/

diff/

DiffInline.js

11 lines

Diff 43249

View Options

resources/celerity/map.php