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

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

Details

Reviewers

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