Skip to content

added delete button with render fix#16

Open
MuhammedJaseemVK wants to merge 1 commit intoTheOpenInnovator:mainfrom
MuhammedJaseemVK:add-delete-button
Open

added delete button with render fix#16
MuhammedJaseemVK wants to merge 1 commit intoTheOpenInnovator:mainfrom
MuhammedJaseemVK:add-delete-button

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