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

John Hill #415

Open
wants to merge 36 commits into
base: main
Choose a base branch
from
Open

John Hill #415

wants to merge 36 commits into from

Conversation

jphill19
Copy link

Completed the project, including all the iterations and the optional content. I tried to add a self challenge by incorporate entry level Typescript with these simple type checks:

  • "noUnusedLocals": true,  
    
  • "noUnusedParameters": true,  
    
  • "noImplicitReturns": true,  
    

If I mad more time, I would add more features to the App, such as deleting saved posters, adjusting my drag code to not be swap, options to create unmotivational posters, etc.

jphill19 and others added 30 commits August 19, 2024 17:49
Added: Validation to the form
… in an array and shoves creates html elements for it
Added funcitonality to save poster button
…incase a user utilizes the escape key to exit
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.

1 participant