Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add trigger when pasting ${} inside a string #38

Open
younesaassila opened this issue Dec 22, 2020 · 2 comments
Open

Add trigger when pasting ${} inside a string #38

younesaassila opened this issue Dec 22, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@younesaassila
Copy link

Hi! Thank you for your extension. It's super useful! I do have a feature suggestion to hopefully make it even better:

The extension currently does not trigger on paste events, thus not converting strings to template strings after pasting a character sequence containing "${}". I think adding a paste event trigger would be a nice addition to fix this issue.

Extension version: 0.4.4
VS Code version: 1.52.1

@younesaassila younesaassila changed the title Add trigger when pasting ${} inside a string Add trigger when pasting ${} inside a string Dec 22, 2020
@meganrogge
Copy link
Owner

Hey! Glad you like it and thanks for the suggestion. I considered doing something for paste, but thought it would be a super rare use case.

Can you give me an example you've encountered where this feature would be useful?

@younesaassila
Copy link
Author

younesaassila commented Dec 22, 2020

I don't have any specific example in mind, but I find myself copy-pasting parts of existing strings with ${} around quite a lot when refactoring, and that's when I notice this lack of support for paste the most, but it's definitely not a huge problem. It would just be a nice addition to make the extension feel more complete.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants
@meganrogge @younesaassila and others