Issue with updating themes after switching #69

Merged
fushiwara-risu merged 1 commit from main into main 2026-02-05 10:57:25 +00:00
fushiwara-risu commented 2026-02-04 15:58:28 +00:00 (Migrated from github.com)

I noticed on my version of mdbook (v0.5.2), the IDs for the theme menu options are all prepended with mdbook-theme-. My simple fix is just to add that to the getElementById call. This fixes the issue during my integration tests.

I noticed on my version of mdbook (v0.5.2), the IDs for the theme menu options are all prepended with `mdbook-theme-`. My simple fix is just to add that to the getElementById call. This fixes the issue during my integration tests.
fushiwara-risu commented 2026-02-04 16:17:50 +00:00 (Migrated from github.com)

Looks like the change happened in v0.5.0 source

Looks like the change happened in v0.5.0 [source](https://github.com/rust-lang/mdBook/blob/b8c90970a63966154b19b4c0a1e4d6ba52fe6375/CHANGELOG.md?plain=1#L116)
badboy (Migrated from github.com) approved these changes 2026-02-05 10:44:22 +00:00
badboy (Migrated from github.com) left a comment

Thanks!

Thanks!
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jer/mdbook-mermaid!69
No description provided.