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

added delete button with render fix #16

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MuhammedJaseemVK
Copy link
Contributor

@TheOpenInnovator Hi. I have added add-delete button with render fix. Kindly review

@TheOpenInnovator
Copy link
Owner

Hey @MuhammedJaseemVK The functionality is great but there are some issues.

  • When the Delete button is pressed, only that Color Palate should be deleted, but here all the palates generated are getting deleted.
  • If required, provide a delete all palates button seperately, where on clicking that and on further confirmation to delete all, then all the Color Palates generated will be deleted.
  • When the Delete button is pressed, ask for a confirmation message & if pressed yes then it should be deleted.

Please Do make the following changes and also if required add any other changes which may benefit the overall website. DO mention all the changes whatever you have made.

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

Successfully merging this pull request may close these issues.

2 participants