-
Notifications
You must be signed in to change notification settings - Fork 0
Navigation with back and resume footer buttons #41
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
Conversation
Stowing arm resume icon: Resizing icon canvas: Repositioning of solo back and resume buttons: Replacing hard coded meal state icon image URL: BackIcon explanation: Simplifying nested if statements in Footer.jsx file: |
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.
Good work -- the code looks nice, and the app looks and functions in a clean way. Approved :)
Describe this pull request. Link to relevant GitHub issues, if any.
This PR includes issue #40 deliverables as detailed in the following:
Locate Plate
andDone Eating
buttons in Bite Acquisition screenTODO: the responsiveness of icon buttons and footer buttons in different devices will be addressed as part of issue Make the App Fully Responsive #9
Explain how this pull request was tested, including but not limited to the below checkmarks.
I tested the implementations by running the app screens in my local Safari browser and ensuring:
Before creating a pull request
npm run format
python3 -m black .
in the top-level of this repositoryBefore merging a pull request
Squash and Merge
)