Skip to content

style(titles): consolidate duplicate .titles__track-lang CSS rule#97

Merged
ScottMorris merged 1 commit into
mainfrom
chore/fix-duplicate-track-lang-css
Jun 24, 2026
Merged

style(titles): consolidate duplicate .titles__track-lang CSS rule#97
ScottMorris merged 1 commit into
mainfrom
chore/fix-duplicate-track-lang-css

Conversation

@ScottMorris

Copy link
Copy Markdown
Contributor

Summary

.titles__track-lang was defined twice in TitlesPage.css: a dead full-width/text-sm block (overridden immediately by the block below it) followed by the intended 48px/mono/text-xs block. Consolidate into one block and move the :focus rule to follow it.

No visual change — the second block was already winning the cascade.

Caught during review of #74.

Test plan

  • pnpm format:check passes
  • Visual spot-check: lang input still renders as a narrow mono code field

The block was defined twice: once with full-width/text-sm properties
(dead code — immediately overridden by the second definition) and once
with the intended 48px/mono/text-xs style. Merge into a single block;
move the :focus rule to follow it.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01L3tSFK3P9vGwbNNtb4AYQG
@ScottMorris

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@github-actions

Copy link
Copy Markdown

Spindle JS tests

233 tests  ±0   233 ✅ ±0   5s ⏱️ ±0s
 24 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 95a718e. ± Comparison against base commit 775680c.

@github-actions

Copy link
Copy Markdown

Spindle Rust tests

221 tests  ±0   221 ✅ ±0   2s ⏱️ -1s
  2 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 95a718e. ± Comparison against base commit 775680c.

@ScottMorris ScottMorris merged commit 759d4d5 into main Jun 24, 2026
8 checks passed
@ScottMorris ScottMorris deleted the chore/fix-duplicate-track-lang-css branch June 24, 2026 18:51
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