Skip to content

Comments

fix: order summary display on mobile [SCSE-204]#72

Merged
chanbakjsd merged 3 commits intomainfrom
fix/SCSE-204
Feb 21, 2023
Merged

fix: order summary display on mobile [SCSE-204]#72
chanbakjsd merged 3 commits intomainfrom
fix/SCSE-204

Conversation

@nicolelst
Copy link
Contributor

@nicolelst nicolelst commented Feb 19, 2023

Display order number/status/order date on individual rows for smaller screens

@linear
Copy link

linear bot commented Feb 19, 2023

SCSE-204 Padding between order summary content is too small on mobile

The order summary page's order number/status/order date should be one row each on small screens.

Feedback:

Padding between order summary content is too small on mobile.

Can refer to the attached pdf (Screenshot taken on iPhone 14)

use chakra grid/breakpoint based on page sizes (see theme)

ntuscse-merch-order-summary.pdf

@vercel
Copy link

vercel bot commented Feb 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
fe ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 20, 2023 at 2:57PM (UTC)

@nicolelst nicolelst requested a review from chanbakjsd February 19, 2023 14:44
Copy link
Contributor

@chanbakjsd chanbakjsd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chanbakjsd chanbakjsd merged commit bea0410 into main Feb 21, 2023
@chanbakjsd chanbakjsd deleted the fix/SCSE-204 branch February 21, 2023 02:45
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 this pull request may close these issues.

2 participants