fix: moved our theme switching from custom attribute theme= to class based.

[adamleithp]

Problem (not really)

• Better CSS specificity control
• More consistent with modern CSS practices
• Easier to extend with additional themes
• Better performance (class selectors are generally faster than attribute selectors)

Changes

TBD

Does this work well for both Cloud and self-hosted?

TBD

How did you test this code?

TDB

[posthog-bot]

Hey @adamleithp! 👋
This pull request seems to contain no description. Please add useful context, rationale, and/or any other information that will help make sense of this change now and in the distant Mars-based future.

[posthog-bot]

📸 UI snapshots have been updated

1002 snapshot changes in total. 0 added, 1002 modified, 0 deleted:

• chromium: 0 added, 948 modified, 0 deleted (diff for shard 1, diff for shard 2)
• webkit: 0 added, 54 modified, 0 deleted (diff for shard 2)

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[github-actions]

Size Change: +25 B (0%)

Total Size: 1.11 MB

ℹ️ View Unchanged

Filename	Size	Change
frontend/dist/toolbar.js	1.11 MB	+25 B (0%)

_{compressed-size-action}

[posthog-bot]

📸 UI snapshots have been updated

486 snapshot changes in total. 0 added, 486 modified, 0 deleted:

• chromium: 0 added, 432 modified, 0 deleted (diff for shard 1, diff for shard 2)
• webkit: 0 added, 54 modified, 0 deleted (diff for shard 2)

Triggered by this commit.

👉 Review this PR's diff of snapshots.

[posthog-bot]

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.