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

Replace insert-text-at-cursor library from examples for newer text-field-edit #478

Open
rodrigoaguilera opened this issue Dec 4, 2024 · 2 comments
Labels
documentation Improvements or additions to documentation

Comments

@rodrigoaguilera
Copy link
Contributor

Using a library that didn't receive any updates for 5 years can be an issue for some folks. At least it was for me so I went on a search for something more maintained.

Seems like
https://www.npmjs.com/package/text-field-edit

Can be a good replacement. Any concerns?
If the maintainers are OK with this I can prepare a PR.

@nolanlawson
Copy link
Owner

This is just an example for documentation, so I don't have a strong opinion on which dependency to use. In fact it'd probably be best to remove the dependency entirely if that's possible.

@nolanlawson nolanlawson added the documentation Improvements or additions to documentation label Dec 4, 2024
@rodrigoaguilera
Copy link
Contributor Author

I agree is just that I wanted the exact same thing as the demo so I want to save others from the search.
I don't think is easy to remove the dependency as the library takes care of many details when performing its focused task.

Please review the PR
#481

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

No branches or pull requests

2 participants