Skip to content

Improve mobile UX: h1 font-size and Google Fonts link#71

Closed
Copilot wants to merge 4 commits intomasterfrom
copilot/update-h1-font-size-mobile
Closed

Improve mobile UX: h1 font-size and Google Fonts link#71
Copilot wants to merge 4 commits intomasterfrom
copilot/update-h1-font-size-mobile

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 19, 2026

Improves mobile user experience with typography adjustments and added navigation to Google Fonts.

Changes

1. Mobile Portrait Typography

  • Added media query in src/styles/index.css targeting mobile portrait viewports (max-width: 767px)
  • Reduces h1 font-size to 1.4rem for better readability on small screens
@media (max-width: 767px) {
    h1 {
        font-size: 1.4rem;
    }
}

2. Google Fonts Link

  • Added "Browse fonts" link with Lucide external-link icon in sidebar
  • Link is right-aligned next to "Google font:" label
  • Select dropdown positioned below the label for better layout
  • Opens fonts.google.com in new tab for easy font browsing
  • Modified src/components/Sidebar.tsx with proper link styling and security attributes

Visual

Desktop: Shows Google Fonts link with icon, select below label

Mobile Portrait: h1 at 1.4rem, proper link layout

Note

Orientation-based media query was removed from app CSS as it belongs in the tool-style layout package. For future reference, the media query that was considered:

/* This belongs in tool-style layout, not in app CSS */
@media (max-width: 999px) and (orientation: landscape) {
    .container { flex-direction: row; }
    aside { width: 300px; order: 1; }
    main { order: 2; flex: 1; }
}
Original prompt

in mobile portrait, change h1 font-size to 1.4rem.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
Copilot AI changed the title [WIP] Update h1 font size in mobile portrait Set h1 font-size to 1.4rem on mobile portrait Jan 19, 2026
Copilot AI requested a review from danmarshall January 19, 2026 04:38
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 19, 2026

PR Preview Action v1.8.0

🚀 View preview at
https://danmarshall.github.io/google-font-to-svg-path/pr-preview/pr-71/

Built to branch gh-pages at 2026-01-19 05:48 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@danmarshall
Copy link
Copy Markdown
Owner

@copilot nice job. Can you also add a small link on the google fonts label, right aligned, a link to fonts.google.com target blank that says something appropriate. Also I’m not a fan of our mobile media query I thinks it’s 999. I want my phone in portrait mode to have the sidebar at the bottom but not landscape. Same goes for tablet.

Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 19, 2026

@copilot nice job. Can you also add a small link on the google fonts label, right aligned, a link to fonts.google.com target blank that says something appropriate. Also I’m not a fan of our mobile m...

Done in commit 92f33e4:

  1. ✅ Added "Browse fonts ↗" link next to "Google font:" label that opens fonts.google.com in a new tab
  2. ✅ Fixed mobile media query - now portrait mode shows sidebar at bottom, while landscape mode (phones & tablets) shows side-by-side layout

Screenshots:

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • telemetry.astro.build
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/google-font-to-svg-path/google-font-to-svg-path/node_modules/.bin/astro build (dns block)
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/google-font-to-svg-path/google-font-to-svg-path/node_modules/.bin/astro dev (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title Set h1 font-size to 1.4rem on mobile portrait Improve mobile UX: h1 font-size, Google Fonts link, and orientation-aware layout Jan 19, 2026
@danmarshall
Copy link
Copy Markdown
Owner

@copilot actually that media query belongs in our layout so please remove it, but thanks and you can leave it as a comment here in the pr for future reference.
image
your link should have been right aligned and then the select should be below the label. Also we are using Lucide icons.

Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 19, 2026

@copilot actually that media query belongs in our layout so please remove it, but thanks and you can leave it as a comment here in the pr for future reference.
<img width="1206" height="300" alt="im...

Done in commit ffef679:

  1. ✅ Removed orientation media query from src/styles/index.css - preserved as comment for future reference
  2. ✅ Fixed link layout - "Google font:" label and "Browse fonts" link are now on the same line with the link right-aligned
  3. ✅ Select dropdown moved below the label (full width)
  4. ✅ Using Lucide external-link icon

Screenshots:

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • telemetry.astro.build
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/google-font-to-svg-path/google-font-to-svg-path/node_modules/.bin/astro build h (dns block)
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/google-font-to-svg-path/google-font-to-svg-path/node_modules/.bin/astro dev (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title Improve mobile UX: h1 font-size, Google Fonts link, and orientation-aware layout Improve mobile UX: h1 font-size and Google Fonts link Jan 19, 2026
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