Skip to content

Conversation

@brysonbw
Copy link
Contributor

Description

Changes

  • Edit homepage-contributor-spotlight__name class styles - prevent name content overflow on mobile screens

Screenshots and screen recordings

Before

contributor-spotlight-overflow-before.mov

After

contributor-spotlight-overflow-after.mov

Related issues and pull requests

@brysonbw brysonbw requested a review from a team as a code owner October 31, 2025 06:23
@brysonbw brysonbw requested a review from caugner October 31, 2025 06:23
Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

I think it should be sufficient to add max-width: 100% to the existing .homepage-contributor-spotlight__name {} rule, where we already set width: max-content.

Setting width: 100% causes the name to take all the space, even if it doesn't need to.

@caugner caugner changed the title fix(homepage-contributor-spotlight): fix contributor spotlight name content overflow fix(homepage-contributor-spotlight): fix content overflow Oct 31, 2025
@brysonbw brysonbw requested a review from caugner November 3, 2025 16:16
@brysonbw brysonbw force-pushed the homepage-contributor-spotlight-name-overflow-fix branch from 321ee9b to 1bd6f5f Compare November 5, 2025 03:03
@brysonbw brysonbw requested a review from caugner November 5, 2025 03:05
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