fix: [M3-10430] - Focus Indicator on MUI Toggle#12988
fix: [M3-10430] - Focus Indicator on MUI Toggle#12988shagufa-akamai merged 3 commits intolinode:developfrom
Conversation
cpathipa
left a comment
There was a problem hiding this comment.
@shagufa-akamai Change set is missing, here is the documentation for changeset.
…ausing visual inconsistency when navigating via keyboard
There was a problem hiding this comment.
Congrats on your first Cloud Manager PR, @shagufa-akamai! 🎉
Thanks for the fix. The focus indicator is a complete circle now.
You can disregard the test failure in CI - looks like unrelated flake. Looks good to merge once CI passes again - rekicking off CI now.
Screen.Recording.2025-10-15.at.10.01.07.AM.mov
Unrelated to this PR: while testing, I noticed that our focus color in light mode for a disabled toggle isn't ideal because it's not fully visible (blends into the white paper background). This isn't a problem in dark mode. Would you mind creating a follow-up M3 ticket to look into fixing this? (If you don't have proper access yet, just let me know, and I can.)
Thanks @mjac0bs. I have created a ticket for the new issue. |
Cloud Manager UI test results🔺 1 failing test on test run #3 ↗︎
Details
TroubleshootingUse this command to re-run the failing tests: pnpm cy:run -s "cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts" |
|||||||||||||||||


Description 📝
Prevent focus indicator on MUI Toggle from being cut off and misaligned
Changes 🔄
List any change(s) relevant to the reviewer.
Scope 🚢
Upon production release, changes in this PR will be visible to:
Target release date 🗓️
Please specify a release date (and environment, if applicable) to guarantee timely review of this PR. If exact date is not known, please approximate and update it as needed.
Preview 📷
Include a screenshot
<img src="" />or video<video src="" />of the change.🔒 Use the Mask Sensitive Data setting for security.
💡 For changes requiring multiple steps to validate, prefer a video for clarity.
How to test 🧪
Prerequisites
(How to setup test environment)
Author Checklists
As an Author, to speed up the review process, I considered 🤔
👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support
As an Author, before moving this PR from Draft to Open, I confirmed ✅