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

Toggle does not work well when it is already marked and selected #18

Closed
jaaviercr opened this issue Jun 13, 2018 · 8 comments
Closed

Toggle does not work well when it is already marked and selected #18

jaaviercr opened this issue Jun 13, 2018 · 8 comments
Labels

Comments

@jaaviercr
Copy link

Toggle does not work well when it is already marked and selected, steps to reproduce it:

  1. Select some text
  2. Toogle highlight
  3. Without loosing selection, toogle highlight again

I think that the correct behaviour would be that the highlight should be cleared but now it does nothing.

Thank you and congratulations for this cool tool 🥇

@ryu1kn
Copy link
Owner

ryu1kn commented Jun 14, 2018

Thanks @jaaviercr , I could reproduce it. Thanks for reporting it!!

Text Marker: v1.1.0
VS Code: v1.24.0
macOS: 10.13.5

@ryan-surrey
Copy link

Any idea of when a fix for this will land?

I've just come across this extension, firstly let me state that its great, i love it, apart from this defect which forces me to have two key bindings to be able to quickly toggle highlighting.

Using v1.3.0

@ryu1kn
Copy link
Owner

ryu1kn commented Jul 18, 2018

Thanks @ryan-surrey for the encouraging comment, also the information how it's affecting your workflow. I'm a bit busy at the moment but hope that I can look into it this weekend 👍

@ryu1kn ryu1kn added the bug label Jul 22, 2018
ryu1kn added a commit that referenced this issue Jul 22, 2018
ryu1kn added a commit that referenced this issue Jul 22, 2018
ryu1kn added a commit that referenced this issue Jul 22, 2018
@ryu1kn
Copy link
Owner

ryu1kn commented Jul 22, 2018

The fix released as v1.3.1. @jaaviercr @ryan-surrey it would be great if you could confirm it's working fine for you 😉

@ryu1kn ryu1kn closed this as completed Jul 22, 2018
@jaaviercr
Copy link
Author

@ryu1kn It works perfect now! Thank you very much! 😄

@ryu1kn
Copy link
Owner

ryu1kn commented Jul 22, 2018

Cool! Thanks for confirming @jaaviercr 👍

@ryan-surrey
Copy link

@ryu1kn, just picked up the latest version, thanks for the fix, its working great!

@ryu1kn
Copy link
Owner

ryu1kn commented Aug 5, 2018

Thanks for confirming it @ryan-surrey ! 👍 (Sorry, seems somehow I missed the notification...)

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

No branches or pull requests

3 participants