Skip to content

css: stop prose stretching the agent picker brand marks - #5714

Open
dimitrieh wants to merge 1 commit into
mainfrom
css/agent-picker-mark-size
Open

css: stop prose stretching the agent picker brand marks#5714
dimitrieh wants to merge 1 commit into
mainfrom
css/agent-picker-mark-size

Conversation

@dimitrieh

Copy link
Copy Markdown
Contributor

Description

The agent picker's brand marks rendered at full column width on any surface that wraps it in a plain .prose. On the changelog entry the ChatGPT square came out 780px.

.prose:not(.prose-natural-size-images) img is (0,2,1) and forces width:100%. .ff-agent-tab img was only (0,1,1), so it lost regardless of source order. The component's styles are deliberately unlayered to beat Tailwind's @layer utilities, but unlayered does not beat a more specific unlayered rule, which is the case that was missed.

Scoping the rule through the card and the tab strip takes it to (0,3,1).

Affected:

Verified all four surfaces at 375, 414, 768, 1024, 1280 and 1536px. Every mark measures 16x16 in all 24 combinations, and the tab strip still wraps correctly at 375px.

Related Issue(s)

Found while reviewing #5671, which adds a fourth surface for this component.

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
  • For blog PRs, an Art Request has been created (instructions)

.prose:not(.prose-natural-size-images) img is (0,2,1) and .ff-agent-tab img
was (0,1,1), so on any surface using a plain .prose the prose rule won and
every mark rendered at width:100% of the column. The ChatGPT square came out
780px on the changelog entry. The docs page was unaffected because its
container carries prose-natural-size-images, and /ai because it renders
outside prose.

Scoping the rule through the card and the tab strip takes it to (0,3,1), so
it holds in prose and out of it.
@netlify

netlify Bot commented Aug 31, 2026

Copy link
Copy Markdown

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit b1894af
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/6a956b81dae58f0008c72034
😎 Deploy Preview https://deploy-preview-5714--flowforge-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 61 (🟢 up 10 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 85 (🔴 down 7 from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant