fix: exclude iframes from focus-visible auto outline in Preflight

[highoncomputers]

Description

Firefox's UA stylesheet already sets iframe:focus-visible { outline-style: none; }, so Preflight's :-moz-focusring { outline: auto; } rule overrides that and applies an unwanted auto outline to focused iframes.

Adding :where(:not(iframe)) to the selector preserves the improved focus ring behavior for all other elements while respecting Firefox's native iframe focus styling.

Fixes #19795

Edit by @RobinMalfait

Test plan

Before:

After:

[greptile-apps]

Confidence Score: 5/5

This looks safe to merge.

• No blocking issues found in the changed code.

_{Reviews (4): Last reviewed commit: "update integration test snapshots" | Re-trigger Greptile}

[greptile-apps]

Preflight Snapshots Stay Stale

This selector changes the generated Preflight CSS, but the checked-in expected outputs still contain the old :-moz-focusring rule. The Preflight and CLI tests that compare generated CSS can fail with a snapshot mismatch until those expected outputs are updated.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: edf3c626-f4ec-46bf-bca5-0d585ad542ea

📥 Commits

Reviewing files that changed from the base of the PR and between 005b8bd and fb662db.

📒 Files selected for processing (2)

• CHANGELOG.md
• integrations/cli/index.test.ts

✅ Files skipped from review due to trivial changes (2)

• CHANGELOG.md
• integrations/cli/index.test.ts

---

Walkthrough

This change updates the Firefox focus-ring selector in preflight.css to :-moz-focusring:where(:not(iframe)), leaving the outline: auto declaration unchanged. The source-map test snapshot and CLI snapshot are updated to match the new selector text and source-range marker, and the changelog adds a corresponding fixed entry.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: excluding iframes from Preflight's Firefox auto-outline rule.
Description check	✅ Passed	The description matches the code changes and explains the iframe focus outline fix accurately.
Linked Issues check	✅ Passed	The PR implements the linked issue's requested selector change to preserve Firefox's native iframe focus styling.
Out of Scope Changes check	✅ Passed	The extra changelog and snapshot updates are directly tied to the iframe focus-outline fix and are in scope.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[greptile-apps]

Snapshot Still Stale The generated Preflight output now contains :-moz-focusring:where(:not(iframe)), but the CLI integration inline snapshot still expects :-moz-focusring {. When that CLI test compares the generated CSS against the inline snapshot, it can still fail with a snapshot mismatch. Please update the remaining inline snapshot along with the checked-in snapshot files.

[greptile-apps]

CLI Snapshot Still Stale

The generated Preflight selector changed here, but the CLI integration inline snapshot still expects :-moz-focusring { for the @import 'tailwindcss' output. When that test compares the generated CSS, it will receive :-moz-focusring:where(:not(iframe)) { and fail the snapshot match. Please update the remaining CLI inline snapshot with this selector change.

[RobinMalfait]

Thanks! Just make sure that next time you run the tests 💪