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

feat: minor improvements and refactoring in Text Replacer Tool #33

Merged
merged 2 commits into from
Mar 6, 2025

Conversation

EugSh1
Copy link
Collaborator

@EugSh1 EugSh1 commented Mar 6, 2025

Hi again!

Following up on my previous pull request regarding issue #10, I’ve made additional updates and improvements:

  • Changed the replaceText method to take all options as arguments.
  • Removed compute from SimpleRadio component.
  • Moved InitialValuesType type and initialValues object to a separate file to avoid Fast Refresh error.
  • Used ToolContent and added usage examples.

These changes address the previous feedback, and I believe everything is now in place for approval. Please let me know if you need any further adjustments.

Looking forward to your feedback! Thanks again!

- Changed the replaceText method to take all options as arguments.
- Removed compute from SimpleRadio component.
- Moved InitialValuesType type and initialValues object to a separate file to avoid Fast Refresh error.
- Used ToolContent and added usage examples.
@iib0011 iib0011 merged commit aa1f97f into iib0011:main Mar 6, 2025
3 checks passed
@iib0011
Copy link
Owner

iib0011 commented Mar 6, 2025

Thank you. You could work on tools like https://onlinetexttools.com/repeat-text or https://onlinetexttools.com/truncate-text
or any other tool you find interesting

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

Successfully merging this pull request may close these issues.

2 participants