-
-
Notifications
You must be signed in to change notification settings - Fork 409
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[HTML-REPORT][BUG]Interactive report missing the Light and Dark mode switch #694
Comments
I would like to join this issue |
Hello @maya5768 , just checking what you mean by "join the issue" ? |
I will try , but I need more information.
|
Thanks for clarifying.
Just run a test with an earlier version, Example: 1.3.20(more or less), you should be able to figure out the Light/Dark mode on the HTML report.
Check the Report related code here. You should find in general how it is building up the reports.
Same as 1. Example:![]() |
@maya5768
|
Hi @authorjapps,
I've successfully added the toggle icon back and tested it locally. I can create the pull request for the fix if it's allowed Kindly advice, thanks! |
@bppdanto-t , thanks for your investigation and more insight on this.
That's amazing. Hope this is not a massive change for you to pull it from bottom left to top right? Please go ahead with the PR. |
@authorjapps Please help to review the PR. I moved the icon to the top right with the same style of the other element besides it. I attached the result in the PR as well. Thank you for this. |
Fixed. |
The earlier version of generate HTML report was having a "toggle" feature for Light/Dark mode towards the bottom left corner.
However, this feature is missing in the latest version.
This change could be due to a recent update in the Extent library, or possibly an unintentional code change that disabled it—though the library update seems more likely. It's also possible that the Extent library now requires a specific flag to enable this feature.
AC1:
Investigate the root cause of this bug and comment in this ticket
AC2;
Fix and raise a PR
Screenshots
Light mode:
Dark mode:
The text was updated successfully, but these errors were encountered: