-
Notifications
You must be signed in to change notification settings - Fork 108
[MBL-19385][All] Fix light mode button disappearing after rotation #3285
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
Conversation
Fixed issue where the "Switch to Light Mode" button in calendar event details would disappear after screen rotation and the theme state would not persist. refs: MBL-19385 affects: Parent, Student, Teacher release note: Fixed light mode button disappearing after screen rotation in calendar event details test plan: 1. Enable dark mode on device 2. Navigate to calendar event with HTML description 3. Verify "Switch to Light Mode" button appears 4. Rotate device - button should remain visible 5. Toggle to light mode, rotate device - light mode should persist 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
adamNagy56
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- In the Parent and Teacher apps, the Light Mode button still disappears as before. It works correctly in the Student app.
See attached video.
Screen_Recording_20251006_223312_Canvas.Parent.mp4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- I'd say this is just 'partially' fix the problem, since the app theme state will persist, but the button itself is still disappearing after rotation change (in Teacher and Parent apps).
calendar-switch-to-light-mode-button-disappear.mp4
📊 Code Coverage Report✅ Student
✅ Teacher
|
kdeakinstructure
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA 👍
There's no "Switch to Dark/Light mode" button at all when we have a To Do item type with description. Should we fix it as well? (That could be in a separate ticket if you think it's a valid issue.)
Fixed the issue where savedHtml was not being updated during configuration changes in ComposeCanvasWebViewWrapper. Changed savedHtml from a simple saved value to a MutableState that can be updated in the update block, ensuring the HTML content stays current during rotation and the light mode button remains visible. refs: MBL-19385
adamNagy56
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA +1
Fixed issue where the "Switch to Light Mode" button in calendar event details would disappear after screen rotation and the theme state would not persist.
refs: MBL-19385
affects: Parent, Student, Teacher
release note: Fixed light mode button disappearing after screen rotation in calendar event details
test plan:
🤖 Generated with Claude Code