Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX: Cursor inserted at the beginning of a paragraph instead of where the click happened #2564

Merged
merged 1 commit into from
Aug 28, 2017

Conversation

tg-ephox
Copy link
Contributor

@tg-ephox tg-ephox commented Aug 28, 2017

fixes #2472

Was only occurring after page load, when the block was first clicked into.

Setting focus on the containing div was causing selection to be lost in Safari. NB contains is inclusive.

@tg-ephox tg-ephox requested a review from youknowriad August 28, 2017 05:11
@tg-ephox tg-ephox self-assigned this Aug 28, 2017
@codecov
Copy link

codecov bot commented Aug 28, 2017

Codecov Report

Merging #2564 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2564   +/-   ##
=======================================
  Coverage   31.07%   31.07%           
=======================================
  Files         174      174           
  Lines        5258     5258           
  Branches      898      898           
=======================================
  Hits         1634     1634           
  Misses       3078     3078           
  Partials      546      546
Impacted Files Coverage Δ
editor/modes/visual-editor/block.js 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 72d4d00...e342060. Read the comment docs.

Copy link
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works great, thanks for the fix.

@tg-ephox tg-ephox merged commit 13a702a into master Aug 28, 2017
@tg-ephox tg-ephox deleted the fix/2472-cursor-inserted-at-beginning branch August 28, 2017 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cursor inserted at the beginning of a paragraph instead of where the click happened
2 participants