Skip to content
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

211-mobile-view-cutoff #212

Merged
merged 1 commit into from
Feb 27, 2024
Merged

211-mobile-view-cutoff #212

merged 1 commit into from
Feb 27, 2024

Conversation

alishaevn
Copy link
Member

@alishaevn alishaevn commented Feb 27, 2024

story

remove unintended white space on mobile

when viewing the webstore on mobile, or inspecting from the browser using a mobile device, there was extra white space alone the right side of the page. turns out that the ItemGroup component was causing it.

we're using a bootstrap grid class that was adding x/y gutters in between each item. this commit removes the horizontal (x) gutter, opting only for the vertical (y) gutter. the page still looks good on desktop too.

demo

before after
mobile image image
desktop image image

when viewing the webstore on mobile, or inspecting from the browser using
a mobile device, there was extra white space alone the right side of
the page. turns out that the ItemGroup component was causing it.

we're using a bootstrap grid class that was adding x/y gutters in between
each item. this commit removes the horizontal (x) gutter, opting only
for the vertical (y) gutter. the page still looks good on desktop too.

- ref: https://getbootstrap.com/docs/5.0/layout/gutters/
- ref: #211
@alishaevn alishaevn linked an issue Feb 27, 2024 that may be closed by this pull request
2 tasks
@alishaevn alishaevn changed the title remove unintended white space on mobile 211-mobile-view-cutoff Feb 27, 2024
Copy link
Collaborator

@labradford labradford left a comment

Choose a reason for hiding this comment

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

I'm having trouble starting up the app, but this is just a css change so I think it will be ok to approve

@alishaevn alishaevn merged commit d78f31d into main Feb 27, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix mobile view cutoff
3 participants