Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Inline Editor: make sure cursor is visible after a change event is received #864

Merged
merged 2 commits into from May 11, 2012

Conversation

gruehle
Copy link
Member

@gruehle gruehle commented May 11, 2012

Fixes issue #752.

When a change event for an inline editor is received, make sure the cursor is visible after all other updates. This fixes two problems:

  1. the cursor is now visible after pasting text that overflows beyond the bottom of the editor
  2. the scroll position of the outer editor was being left in an inconsistent (and incorrect) state

@redmunds
Copy link
Contributor

reviewing

@redmunds
Copy link
Contributor

Maybe it would be a good time to fill in the comment above the _updateRelatedContainer() function?

@gruehle
Copy link
Member Author

gruehle commented May 11, 2012

Comment added.

@redmunds
Copy link
Contributor

Looks good. Verified fix on Win7.

redmunds added a commit that referenced this pull request May 11, 2012
Inline Editor: make sure cursor is visible after a change event is received
@redmunds redmunds merged commit 4dad77f into master May 11, 2012
gideonthomas pushed a commit to gideonthomas/brackets that referenced this pull request Aug 31, 2017
this updates the code such that it aligns with the code comment explaining what's supposed to happen.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants