-
Notifications
You must be signed in to change notification settings - Fork 49
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
Comments
I'm using Atom version 0.113.0, and Sublime Style Column Selection 1.1.0. |
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:
It might also be to me copy+pasting the selection code from atom itself. |
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? |
@Zren Nice fork! |
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 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. |
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. |
Yeah, I don't claim to be mimicking lightpaper. |
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. |
Here's the expected behavior (using LightPaper):
The text was updated successfully, but these errors were encountered: