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

12. As a user, I want to be able to delete items from my shopping list so that my list isn’t cluttered with items I don’t want to buy in the future #12

Closed
3 tasks done
fullybaked opened this issue Aug 7, 2024 · 0 comments · Fixed by #31
Assignees

Comments

@fullybaked
Copy link
Contributor

fullybaked commented Aug 7, 2024

Summary

Users might make a mistake when entering an item, or may decide they won’t be buying a certain item again in the future. Allowing them to delete list items will help them keep their lists tidy.

Acceptance criteria

  • The ListItem component renders a button that allows the user to delete an item from their list when clicked
  • Clicking the delete button prompts the user to confirm that they really want to delete the item
  • The deleteItem function in api/firebase.js has been filled out, and deletes the item from the Firestore database

Notes:

  • Activating the “Delete” button should show a confirmation dialog. We want to give users a chance to confirm “destructive” actions.
  • It might be tempting to create your own modal dialog component to handle confirmation, but modals are fraught with accessibility concerns, so consider using a simple JavaScript confirm dialog.
shuveksha-tuladhar added a commit that referenced this issue Sep 17, 2024
…e JS confirm dialog and shows user a message accordingly.
shuveksha-tuladhar added a commit that referenced this issue Sep 19, 2024
stacy-tech added a commit that referenced this issue Sep 21, 2024
Issue #12: Delete Items from Shopping List
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 a pull request may close this issue.

3 participants