Skip to content

fix(desktop): Compile annotation CSS in the preload bundler - #83

Closed
mwolson wants to merge 1 commit into
contrib/native-stack-patch-hunk-orderfrom
contrib/annotation-css-bundler-plugin
Closed

mwolson wants to merge 1 commit into
contrib/native-stack-patch-hunk-orderfrom
contrib/annotation-css-bundler-plugin

Conversation

@mwolson

@mwolson mwolson commented Oct 10, 2026

Copy link
Copy Markdown
Owner

What Changed

Carries pingdotgg#17850 annotation-css-virtual-module. The preview annotation stylesheet is compiled by a bundler virtual module (virtual:preview-annotation-css) in the picker preload build, with watch rebuilds, instead of a script that wrote AnnotationStyles.generated.ts into the source tree. The legacy generated file and script are removed, and the stylesheet the app embeds is unchanged.

Why

The checked-in generated file had drifted from its generator, so every desktop build rewrote it and left the Nightly tree dirty, which blocked the next build. This replaces the earlier personal layer #79 annotation-styles-regenerated. Drop this layer at the refresh after pingdotgg#17850 merges upstream.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Carries pingdotgg#17850 annotation-css-virtual-module (two commits
squashed): the preview annotation stylesheet is compiled by a bundler
virtual module with watch rebuilds instead of a checked-in generated file,
so desktop builds no longer rewrite tracked sources. Drop this layer at the
refresh after pingdotgg#17850 annotation-css-virtual-module merges upstream.

Co-authored-by: Mike Olson <1510193+mwolson@users.noreply.github.com>
@mwolson
mwolson added this pull request to stack #84 October 10, 2026 16:45
@mwolson
mwolson removed this pull request from stack #84 October 15, 2026 01:56
@mwolson

mwolson commented Oct 15, 2026

Copy link
Copy Markdown
Owner Author

Dropped at the 2026-10-14 refresh onto upstream 40a7908: merged upstream pingdotgg#17838 annotation-style-sync keeps the generated annotation CSS current at this pin, so this carry of pingdotgg#17850 annotation-css-virtual-module is no longer needed. Re-carry it if a Nightly build leaves AnnotationStyles.generated.ts dirty.

@mwolson mwolson closed this Oct 15, 2026
@mwolson
mwolson deleted the contrib/annotation-css-bundler-plugin branch October 15, 2026 01:56
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