Skip to content

No visual indicator (Mimick lightpaper's '+' ?) #12

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

Open
getaaron opened this issue Jul 9, 2014 · 8 comments
Open

No visual indicator (Mimick lightpaper's '+' ?) #12

getaaron opened this issue Jul 9, 2014 · 8 comments

Comments

@getaaron
Copy link

getaaron commented Jul 9, 2014

  1. The arrow should turn into a "+" icon when I hold alt
  2. I can't actually select anything because the text jumps around:

jumpy selection

Here's the expected behavior (using LightPaper):

jumpy selection - lightpaper

@getaaron
Copy link
Author

getaaron commented Jul 9, 2014

I'm using Atom version 0.113.0, and Sublime Style Column Selection 1.1.0.

@Zren
Copy link

Zren commented Jul 11, 2014

Probably need to do this: https://github.com/Zren/atom-sublime-select/blob/master/lib/sublime-select.coffee#L45-L47 as I didn't get any jumping after that commit. Basically you need the:

+ editorView.scrollLeft()

It might also be to me copy+pasting the selection code from atom itself.

@bigfive
Copy link
Owner

bigfive commented Jul 15, 2014

I think that was an issue with that version of Atom. I had that issue for a while but it seemed to have disappeared with the latest release.

Do you still have the issue after updating atom?

@bigfive
Copy link
Owner

bigfive commented Jul 15, 2014

@Zren Nice fork!
Any pull requests you wanna share?

@Zren
Copy link

Zren commented Jul 15, 2014

My fork only binds the Middle mouse button to the cursor. I copy pasta'd the selection code from atom's source. I still have to run through it's performance though.

One thing I might PR is how you require the emissary dependency. It should be already loaded & cached by atom, and loading another instance for each package is silly. I've been forking a few packages to increase their load/activation times.

I'm doing

{Subscriber} = require atom.packages.resourcePath + '/node_modules/emissary'

and it seems to work.

It also drops load times from 47ms to 16ms.

This pattern isn't widespread however. Probably because it might break things atom changes around.

@getaaron
Copy link
Author

Updating to Atom 0.115 resolved the jumpy selection issue for me. However, the arrow should turn into a "+" icon when I hold alt, and it doesn't, so I'm leaving this issue open for now.

@bigfive
Copy link
Owner

bigfive commented Jul 28, 2014

Yeah, I don't claim to be mimicking lightpaper.
Its a good feature request tho

@bigfive bigfive changed the title Jumpy scrolling, and no visual indicator No visual indicator (Mimick lightpaper's '+' ?) Oct 10, 2014
sadovnychyi added a commit to sadovnychyi/atom-sublime-select that referenced this issue Aug 25, 2015
@sadovnychyi
Copy link
Contributor

I got a fix for "The arrow should turn into a "+" icon when I hold alt" in #71. Please take a look. Ideas are welcome.

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

No branches or pull requests

4 participants