-
Notifications
You must be signed in to change notification settings - Fork 0
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
Edit individual list items. #54
Conversation
Visit the preview URL for this PR (updated for commit b8d5d5c): https://tcl-79-smart-shopping-list--pr54-hm-edit-listitem-cezd4csr.web.app (expires Sun, 20 Oct 2024 14:58:59 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: d91d9ddbda780208241c52942f544acf8e81407a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love this additional feature!
Small note I think the pencil and trash can should be two different colours but not necessarily the pinks and greens we use throughout the app(cause there is a lot of colour going on) Maybe just black/dark grey/dark slate on Light mode and a a light grey/slate on Dark mode? I don't know🤷🏾♀️
…te inactive share button style to match delete button.
I did the following in my recent commits:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work and really nice feature! Well done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
Users can now edit their item details directly through the "Edit Item" form. This feature allows them to modify the item name, update the next purchase date, and adjust the quantity.
Related Issue
closes #49
Acceptance Criteria
Type of Changes
Feature
Updates
Before
After
Testing Steps / QA Criteria
git pull
andgit checkout hm-edit-listItem
.npm start
.