Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Load the user's preferred color scheme if the URL contains an invalid theme #2007

Merged
merged 1 commit into from
Apr 3, 2025

Conversation

0xh3xa
Copy link
Contributor

@0xh3xa 0xh3xa commented Mar 29, 2025

What:

To Reproduce

visit: https://gchq.github.io/CyberChef?theme=foo where foo is not a valid theme name.
It does not occur when foo is a valid theme name or for https://gchq.github.io/CyberChef

Expected behaviour

Cyberchef should probably give an "invalid theme" message or load with the default theme.

@0xh3xa 0xh3xa changed the title Load the userpreferred color scheme if the URL contains an invalid theme Load the user's preferred color scheme if the URL contains an invalid theme Mar 29, 2025
@a3957273
Copy link
Member

a3957273 commented Apr 3, 2025

Looks good to me, thanks so much! :)

@a3957273 a3957273 merged commit 14472d3 into gchq:master Apr 3, 2025
4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug report: invalid theme gives TypeError: t.querySelector(...) is null
2 participants