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.

Sign up for GitHub

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

Jump to bottom

Unify line boundary detection #225

Closed

peitschie opened this issue Nov 28, 2013 · 1 comment

Assignees

Labels

cleanup

Milestone

Contributor

peitschie commented Nov 28, 2013

SelectionMover has multiple algorithms that attempt to detect line boundaries. These should be de-duped.

SelectionMover.countLineSteps
SelectionMover.countStepsToLineBoundary

I'd suggest this is a necessary task for fixing issue #224

peitschie self-assigned this

peitschie mentioned this issue

Rewrite of line boundary movement actions #609

Merged

peitschie mentioned this issue

Remove superseded SelectionMover class #730

Merged

Contributor Author

peitschie commented Aug 5, 2014

The SelectionMover class has now been removed completely, and the line boundary detection has been unified to a common set of filters.

peitschie closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment