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

Revamp Filler Text page #28

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Revamp Filler Text page #28

wants to merge 5 commits into from

Conversation

g1oom
Copy link
Member

@g1oom g1oom commented Jan 26, 2024

close #23

  • change form inputs to allow typing of word and paragraph count. text will only generate after proper validation of values
  • add logic to limit word count between 1 and 5000, paragraph count between 1 and 100, and word count cannot be less than paragraph count
  • overhaul text generator to ensure all edge cases are considered
  • add copy button to allow copying of text easily to clipboard

albert-ong-wei-jun and others added 3 commits October 27, 2023 00:49
* enable word count to decrement below 500
* fix bug where website crashes when delete counts input
* change form inputs to allow typing of word and paragraph count. text will only generate after proper validation of values
* add logic to limit word count between 1 and 5000, paragraph count between 1 and 100, and word count cannot be less than paragraph count
* overhaul text generator to ensure all edge cases are considered
* add copy button to allow copying of text easily to clipboard
* ensure more equal distribution of words per paragraph when the initially generated text has fewer words than required word count
@g1oom g1oom self-assigned this Jan 26, 2024
@g1oom
Copy link
Member Author

g1oom commented Jan 29, 2024

close #24

@g1oom g1oom linked an issue Jan 29, 2024 that may be closed by this pull request
@g1oom
Copy link
Member Author

g1oom commented Feb 6, 2024

changes made

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.

Stop focusing on <textarea> of generated words Add a word & paragraph count limit
2 participants