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

Make scroll follow the cursor #16

Closed
michael opened this issue Aug 25, 2015 · 6 comments
Closed

Make scroll follow the cursor #16

michael opened this issue Aug 25, 2015 · 6 comments
Milestone

Comments

@michael
Copy link
Member

michael commented Aug 25, 2015

This is really needed for a serious editing experience. I will work on a scrollToCursor/Selection implementation and then we connect it with the Surface.

@michael michael added this to the Beta 2 milestone Aug 25, 2015
@michael
Copy link
Member Author

michael commented Sep 2, 2015

If only this would work for contenteditable too: http://jsfiddle.net/2r2fpekx/12/

@obuchtala
Copy link
Member

One problem is that a collapsed selection does not have a bounding rect.
This describes a hack to get such a rect by injecting a span temporarily: http://ask-wenda.appspot.com/r/5707702298738688/5648640/know-when-caret-goes-outside-of-contenteditable-scrollable-div

@obuchtala
Copy link
Member

When we start to render the selection ourselves we will have everything to get this done.

@obuchtala
Copy link
Member

We can't do that yet, as we need to render selections always for that.

Shifting to Beta 3

@obuchtala obuchtala modified the milestones: Beta 3, Beta 2 Nov 27, 2015
@michael michael modified the milestones: Beta 3, Beta 4 Dec 9, 2015
sunu pushed a commit to sunu/substance that referenced this issue May 19, 2016
@obuchtala
Copy link
Member

While we have the methods now, we still need to integrate it. Shifting to beta 5.

@obuchtala obuchtala modified the milestones: Beta 5, Beta 4 Jun 13, 2016
@michael michael modified the milestones: Beta 5, Beta 6 Sep 29, 2016
@michael
Copy link
Member Author

michael commented Nov 18, 2016

Fixed via #882

@michael michael closed this as completed Nov 18, 2016
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

No branches or pull requests

2 participants