Skip to content

Conversation

Copy link

Copilot AI commented Sep 9, 2025

The external link icon displayed after target="_blank" links was using a fixed size of 0.6rem, causing it to appear disproportionately large on mobile devices and not scale with different text sizes.

Problem

The icon had a fixed height regardless of the text size it was attached to, making it look oversized especially on phones where text might be smaller or when used with different font sizes throughout the interface.

Solution

Changed the SVG height from height="0.6rem" to height="1em" in the CSS rule for target blank icons. This makes the icon scale relative to the font size of the text it's attached to, ensuring proportional sizing across all devices and text sizes.

Technical Details

The change was made in /src/styles/main.css in the CSS rule:

a[target="_blank"]:not(:has(h5))::after,
a[target="_blank"] > h5::after

The fix ensures the icon now properly scales with:

  • Small text (12px) - icon scales down appropriately
  • Normal text (16px) - maintains good proportion
  • Large text (24px+) - scales up to match
  • Mobile viewports - no longer appears oversized

The change is minimal and maintains all existing functionality while improving the visual consistency across different screen sizes and text configurations.

Fixes #2.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copy link
Author

Copilot AI commented Sep 9, 2025

@PatrickMatthiesen 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Co-authored-by: PatrickMatthiesen <43612965+PatrickMatthiesen@users.noreply.github.com>
Copilot AI changed the title [WIP] Target blank icon is very large on phone Fix target blank icon scaling to match text size on mobile devices Sep 9, 2025
Copy link

@PatrickMatthiesen PatrickMatthiesen left a comment

Choose a reason for hiding this comment

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

LGTM

@PatrickMatthiesen PatrickMatthiesen marked this pull request as ready for review September 14, 2025 08:37
@PatrickMatthiesen PatrickMatthiesen merged commit 765537f into main Sep 14, 2025
1 check 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.

Target blank icon is very large on phone

2 participants