Emit resize events after making document changes during inline editing
ClosedPublic
Actions

Authored by epriestley on May 16 2017, 2:13 PM.

Tags

None

Referenced Files

	F19666089: D17904.diff
	Sat, Feb 7, 2:30 PM

	F19566925: D17904.id43063.diff
	Sat, Jan 31, 8:22 PM

	F19099687: D17904.id.diff
	Dec 4 2025, 8:50 PM

	F19091715: D17904.diff
	Dec 3 2025, 3:11 PM

	F19026773: D17904.id43074.diff
	Nov 24 2025, 12:16 PM

	F18861380: D17904.diff
	Nov 2 2025, 3:16 PM

	F18829242: D17904.id43063.diff
	Oct 24 2025, 8:42 PM

	F18823532: D17904.id43074.diff
	Oct 23 2025, 12:27 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T12633: Interacting with the InlineEditor does not resize or reposition the keyboard focus reticle
T12634: Keyboard focus reticle is generally oblivious to the world around it

Commits

rP1493f08272dc: Emit resize events after making document changes during inline editing

Summary

Ref T12634. Fixes T12633. These events allow the keyboard reticle to resize properly.

(I expect to possibly hide/disable the reticle in the future during edits, but at least make the behavior sensible for now.)

Test Plan

Used "n" to select a block.
Clicked a line number in that block to start a new inline comment.
Saw reticle resize properly.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 16 2017, 2:13 PM

Harbormaster completed remote builds in B16991: Diff 43063.May 16 2017, 2:15 PM

chad accepted this revision.May 16 2017, 2:32 PM

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

Closed by commit rP1493f08272dc: Emit resize events after making document changes during inline editing (authored by epriestley, committed by epriestley). · Explain WhyMay 16 2017, 3:22 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

6 lines

Diff 43074

View Options

resources/celerity/map.php