Skip to content

fix(desktop): un-break .btn-arc — '*/' inside a CSS comment ended it early (extract #59352) - #77662

Merged
kshitijk4poor merged 1 commit into
NousResearch:mainfrom
kshitijk4poor:salvage/59352-css-comment-fix
Aug 3, 2026
Merged

fix(desktop): un-break .btn-arc — '*/' inside a CSS comment ended it early (extract #59352)#77662
kshitijk4poor merged 1 commit into
NousResearch:mainfrom
kshitijk4poor:salvage/59352-css-comment-fix

Conversation

@kshitijk4poor

Copy link
Copy Markdown
Collaborator

Extract-salvage of the one alive piece of #59352 (author @rerdi92, authorship preserved): the comment above .btn-arc contains 'bg-*/' whose */ terminates the comment early, leaving an invalid prelude that can drop the whole rule. Rewords the comment. The PR's other pieces (month-stale full-file icons.ts rewrite that would drop 10+ live icons; chunkSizeWarningLimit bump that masks rather than fixes) were not salvaged. Closes #59352.

…ended it early

The comment above .btn-arc contained 'bg-*/', whose */ terminated the comment mid-sentence, leaving 'text-* variant utilities. */ .btn-arc {' as an invalid prelude — CSS error recovery can drop the whole .btn-arc rule. Reword so no */ appears inside the comment.

Extracted from NousResearch#59352 by @rerdi92 (the rest of that PR — a month-stale icons.ts rewrite and a chunk-size warning-ceiling bump — is superseded/masking and was not salvaged).
@kshitijk4poor
kshitijk4poor enabled auto-merge (rebase) August 3, 2026 12:40
@kshitijk4poor
kshitijk4poor merged commit 1306ac0 into NousResearch:main Aug 3, 2026
31 checks passed
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/desktop Electron desktop app (apps/desktop/*) labels Aug 3, 2026
@kshitijk4poor
kshitijk4poor deleted the salvage/59352-css-comment-fix branch August 5, 2026 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/desktop Electron desktop app (apps/desktop/*) P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants