Merged
Conversation
added 4 commits
February 23, 2026 13:53
…ry prefs/restrictions
Pranav-Batra
temporarily deployed
to
staging-651
GitHub Actions
Inactive
LexTruong
requested changes
Feb 28, 2026
Collaborator
LexTruong
left a comment
LexTruong
left a comment
There was a problem hiding this comment.
Looks great! UI/UX is still finalizing the look of the conflict indicators, so we're not too worried about the styling. But what you have so far looks pretty nice.
- Please add an indicator on the meal dialog/drawers for incompatibility.
- Remove unnecessary comments.
Pranav-Batra
temporarily deployed
to
staging-651
GitHub Actions
Inactive
LexTruong
requested changes
Mar 10, 2026
Collaborator
LexTruong
left a comment
LexTruong
left a comment
There was a problem hiding this comment.
Hey Pranav, great work! Just a few touchups are needed now that the Figma has been finalized.
- Match the styling for the badges on meal dialog/drawers. Note that they show what specific preference/allergen is problematic in that dish. Use
MUIicons for the exclamation point icon. - I like the slightly greyed-out look on incompatible meal cards! Please change the "Conflict" badge on them to an icon though.
- Make sure that warning icon also shows on meal cards for mobile screens.
added 4 commits
March 30, 2026 09:19
…ssc/PeterPlate into 640/dietary-preferences-and-allergens
Pranav-Batra
temporarily deployed
to
staging-651
GitHub Actions
Inactive
LexTruong
requested changes
Apr 5, 2026
Collaborator
There was a problem hiding this comment.
Hey Pranav, a few more changes are needed on this one.
- The warning icon isn't quite right on the food cards. Also, the icon should be placed immediately to the right of the food name instead of in the top right of the card.
- The warning icon isn't showing correctly on the food dialog (looks like an exclamation point instead of an icon).
- Also, please verify you're using the right shade of red for the warning icons on both the food card and dialogs. According to the Figma, it should be tailwind red 600.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Users can see their dietary prefs/allergens in menu, see which meals conflict with their prefs, and view only meals that agree with their prefs
Changes
Removed hardcoded prefs/allergens on user menu with their actual prefs/allergens
Added red warning badge on meals that do not meet user preferences(either contain an allergen or dont meet dietary prefs)
Added "show preferences only" button to toggle the restaurant pages to only show meals that match user dietary preferences
Closes #640