-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
#292 Improved mobile styling and item card layouts #323
Conversation
✅ Deploy Preview for cool-creponne-3e1272 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
This is looking great!
I didn't really annotate any particular lines but one thing that came up is that the reserved
label is showing only when we look at an item in our profile page (so in DonatedItemsList.tsx
but not on the search page.
We also have a ternary in the image src attribute for ItemCard that won't really do anything because we have enforced the upload of an image when an item is added - that's some legacy code but worth cleaning up while we're at it.
…lay the reserved UI Relates #292
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.
Developers Developers!
It's looking swanky, thank you for updating this!
Description
**Closes #292 **
This PR expands a bit on the issue and seeks to resolve a few layout issue with the app, specifically with the mobile experience. It seeks primarily to deal with stying issue on the search and about routes.
UI changes
Tests
Story files updated to reflect new information being displayed as well as show postage covered state.