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.

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

Multi-line remote selection doesn't match local selection #23

Closed
aldendaniels opened this issue Jun 19, 2013 · 1 comment
Closed

Multi-line remote selection doesn't match local selection #23

aldendaniels opened this issue Jun 19, 2013 · 1 comment

Comments

@aldendaniels
Copy link
Owner

Remote selection:
screen shot 2013-06-19 at 12 23 35 am

Local selection:
screen shot 2013-06-19 at 12 24 19 am

We have a couple of options:

  1. Always use full line selection
  2. Always use text selection
  3. Decide it's OK for these to be different.

It looks like the latest ACE build that we just updated to added support for line selections per https://github.com/ajaxorg/ace-builds/blob/master/ChangeLog.txt.
willerwin pushed a commit that referenced this issue Jul 9, 2013
#23: Use text selection
227b2c7 
This was the easiest solution and made the most sense to me. It could
possibly be just as easy to change for remote users but I didn't look
too hard. If anyone has any passion for full line selection, let's talk.
@josiahdaniels
Copy link
Collaborator

Looks good to me. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@josiahdaniels @aldendaniels