Skip to content

Support brightness override in a11y page #9898

Merged
hannah-hyj merged 3 commits into
flutter:masterfrom
hannah-hyj:a11y_brightness
Jul 17, 2026
Merged

Support brightness override in a11y page #9898
hannah-hyj merged 3 commits into
flutter:masterfrom
hannah-hyj:a11y_brightness

Conversation

@hannah-hyj

@hannah-hyj hannah-hyj commented Jul 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

tracking issue: #9893

Screen.Recording.2026-07-16.at.13.41.32.mov

the ext.flutter.brightnessOverride service extension and VM service support to Flutter was added a while ago in PR #59571

Pre-launch Checklist

General checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I updated/added relevant documentation (doc comments with ///).

Issues checklist

Tests checklist

  • I added new tests to check the change I am making...
  • OR there is a reason for not adding tests, which I explained in the PR description.

AI-tooling checklist

  • I did not use any AI tooling in creating this PR.
  • OR I did use AI tooling, and...
    • I read the AI contributions guidelines and agree to follow them.
    • I reviewed all AI-generated code before opening this PR.
    • I understand and am able to discuss the code in this PR.
    • I have verifed the accuracy of any AI-generated text included in the PR description.
    • I commit to verifying the accuracy of any AI-generated code or text that I upload in response to review comments.

Feature-change checklist

  • This PR does not change the DevTools UI or behavior and...
    • I added the release-notes-not-required label or left a comment requesting the label be added.
  • OR this PR does change the DevTools UI or behavior and...
    • I added an entry to packages/devtools_app/release_notes/NEXT_RELEASE_NOTES.md.
    • I included before/after screenshots and/or a GIF demo of the new UI to my PR description.
    • I ran the DevTools app locally to manually verify my changes.

build.yaml badge

If you need help, consider asking for help on Discord.

@hannah-hyj
hannah-hyj requested a review from elliette July 16, 2026 20:41
@hannah-hyj
hannah-hyj requested review from a team and kenzieschmoll as code owners July 16, 2026 20:41
@hannah-hyj
hannah-hyj requested review from bkonyi and removed request for a team July 16, 2026 20:41
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Jul 16, 2026
View reviewed changes

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request implements the VM service extension integration for overriding brightness in the accessibility controls. It defines the brightnessMode service extension, updates the AccessibilityController to listen to device-side brightness changes, and sends updates back to the VM service when changed via the DevTools UI. Additionally, it adds corresponding analytics constants and comprehensive tests. The reviewer raised a concern regarding potential feedback loops and redundant VM service calls when brightness changes originate from the device itself, suggesting a boolean guard to prevent unnecessary updates.

Comment thread packages/devtools_app/lib/src/screens/accessibility/accessibility_controller.dart
@hannah-hyj
1
cd275cb 
Update overrides_pane.dart

Update accessibility_controller.dart

tests

Update service_extensions.dart
kenzieschmoll
kenzieschmoll reviewed Jul 17, 2026
View reviewed changes

@kenzieschmoll kenzieschmoll left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like this PR is missing the UI code to add the dropdown. Can you include that in this PR?

Comment thread packages/devtools_app/lib/src/screens/accessibility/accessibility_controller.dart Outdated
Comment thread packages/devtools_app/lib/src/screens/accessibility/accessibility_controller.dart Outdated
Comment thread packages/devtools_app/lib/src/service/service_extensions.dart Outdated
Comment thread packages/devtools_app/test/screens/accessibility/accessibility_screen_test.dart Outdated
elliette
elliette reviewed Jul 17, 2026
View reviewed changes
Comment thread packages/devtools_app/lib/src/screens/accessibility/accessibility_controller.dart Outdated
@hannah-hyj

hannah-hyj commented Jul 17, 2026

Copy link
Copy Markdown
Member Author

@kenzieschmoll The UI code was merged in #9892, so this PR is just to add functions

@hannah-hyj hannah-hyj mentioned this pull request Jul 17, 2026
Open
10 tasks
@hannah-hyj hannah-hyj added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 17, 2026
@hannah-hyj
hannah-hyj merged commit e41c353 into flutter:master Jul 17, 2026
94 of 95 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kenzieschmoll kenzieschmoll kenzieschmoll approved these changes
@bkonyi bkonyi Awaiting requested review from bkonyi bkonyi is a code owner automatically assigned from flutter/devtools-reviewers
@elliette elliette Awaiting requested review from elliette
+1 more reviewer
@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

autosubmit Merge PR when tree becomes green via auto submit App release-notes-not-required

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hannah-hyj @elliette @kenzieschmoll