Differential D21248

Allow users to create inline comments by directly selecting text directly
ClosedPublic
Actions

Authored by epriestley on May 13 2020, 8:17 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Feb 1, 6:49 AM

	Unknown Object (File)
	Tue, Jan 28, 9:50 AM

	Unknown Object (File)
	Sat, Jan 25, 3:58 PM

	Unknown Object (File)
	Sat, Jan 25, 7:39 AM

	Unknown Object (File)
	Sat, Jan 25, 7:39 AM

	Unknown Object (File)
	Sat, Jan 25, 7:39 AM

	Unknown Object (File)
	Wed, Jan 22, 4:39 AM

	Unknown Object (File)
	Tue, Jan 21, 2:04 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13513: Make schema changes to inlines to prepare for new inline features
Commits: rP42378ea393fe: Allow users to create inline comments by directly selecting text directly

Summary

Ref T13513. Support direct text selection for inlines. This is currently just an alternate way to get to the same place as using line numbers, but can preserve offset/range information in the future.

Test Plan

Selected some text, hit "c", clicked "New Inline Comment", got sensible comments on both sides of a diff in Safari, Chrome, and (with limitations) Firefox.
Caveats: no unified support, doesn't work across lines in Firefox.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 13 2020, 8:17 PM

Harbormaster completed remote builds in B24415: Diff 50598.May 13 2020, 8:18 PM

epriestley requested review of this revision.May 13 2020, 8:18 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 14 2020, 12:15 AM

Closed by commit rP42378ea393fe: Allow users to create inline comments by directly selecting text directly (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP42378ea393fe: Allow users to create inline comments by directly selecting text directly.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

src/

applications/

differential/

view/

DifferentialChangesetListView.php

8 lines

webroot/

rsrc/

js/

application/

diff/

DiffChangesetList.js

298 lines

core/

behavior-oncopy.js

2 lines

phuix/

PHUIXDropdownMenu.js

84 lines

Diff 50603

resources/celerity/map.php

Loading...

src/applications/differential/view/DifferentialChangesetListView.php

Loading...

webroot/rsrc/js/application/diff/DiffChangesetList.js

Loading...

webroot/rsrc/js/core/behavior-oncopy.js

Loading...

webroot/rsrc/js/phuix/PHUIXDropdownMenu.js

Loading...