Skip to content

fix mobile sizing#50

Merged
alanthai merged 1 commit into
mainfrom
athai/fix-mobile
Mar 30, 2026
Merged

fix mobile sizing#50
alanthai merged 1 commit into
mainfrom
athai/fix-mobile

Conversation

@alanthai
Copy link
Copy Markdown
Contributor

@alanthai alanthai commented Mar 27, 2026

Screenshot 2026-03-27 at 12 19 44 PM

TODO

  • Explore source sets to serve lower bitesize images

@alanthai alanthai requested review from Copilot and davidminin March 27, 2026 16:18
@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented Mar 27, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adjusts typography and top navigation styling to improve mobile layout/sizing in the client UI.

Changes:

  • Fix heading typography by using non-zero line-height and resetting heading margins.
  • Make the top nav brand responsive by swapping between full logo and icon on smaller screens.
  • Tweak mobile spacing/layout in top nav and section grid.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
client/src/design/heading/Heading.module.css Updates heading line-height and removes default heading margins.
client/src/components/top-nav/TopNav.tsx Renders both full logo and icon logo for responsive display.
client/src/components/top-nav/TopNav.module.css Adds responsive padding/logo visibility rules and tighter link spacing for mobile.
client/src/components/section/Section.module.css Increases mobile gap between section columns/rows.
client/src/assets/bolt.svg Adds new bolt icon asset for the mobile nav brand.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread client/src/components/top-nav/TopNav.module.css Outdated
Comment thread client/src/design/heading/Heading.module.css
@alanthai alanthai merged commit 90b11b7 into main Mar 30, 2026
5 checks passed
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.

3 participants