Skip to content

refactor: Remove material imports from editable_text_scribble_test, editable_text_scribe_test, page_route_builder_test, radio_group_test, semantics_debugger_test, range_maintaining_scroll_physics_test, two_dimensional_scroll_view_test, routes_test, text_selection_test, selectable_region_test and text_golden_test#184798

Merged
auto-submit[bot] merged 5 commits into
flutter:masterfrom
rkishan516:simple-cross-imports-8
Apr 24, 2026

Conversation

@rkishan516

@rkishan516 rkishan516 commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

This PR removes Material imports from

  • editable_text_scribble_test
  • editable_text_scribe_test
  • page_route_builder_test
  • radio_group_test
  • semantics_debugger_test
  • range_maintaining_scroll_physics_test
  • two_dimensional_scroll_view_test
  • routes_test
  • text_selection_test
  • selectable_region_test
  • text_golden_test

part of: #177415

Pre-launch 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 and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
  • I signed the [CLA].
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
  • All existing and new tests are passing.

@github-actions github-actions Bot added a: text input Entering text in a text field or keyboard related problems framework flutter/packages/flutter repository. See also f: labels. a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) f: scrolling Viewports, list views, slivers, etc. f: routes Navigator, Router, and related APIs. labels Apr 9, 2026
@rkishan516 rkishan516 added the CICD Run CI/CD label Apr 9, 2026
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

Gemini is experiencing higher than usual traffic and was unable to create the review. Please try again in a few hours by commenting /gemini review.

@rkishan516 rkishan516 force-pushed the simple-cross-imports-8 branch from b175749 to 5abc42c Compare April 9, 2026 08:20
@github-actions github-actions Bot removed the CICD Run CI/CD label Apr 9, 2026
@rkishan516 rkishan516 added the CICD Run CI/CD label Apr 9, 2026
@rkishan516 rkishan516 force-pushed the simple-cross-imports-8 branch from 5abc42c to 3f40857 Compare April 9, 2026 16:07
@github-actions github-actions Bot removed the CICD Run CI/CD label Apr 9, 2026
@rkishan516 rkishan516 added the CICD Run CI/CD label Apr 9, 2026
@rkishan516 rkishan516 force-pushed the simple-cross-imports-8 branch from 3f40857 to e4c9a80 Compare April 10, 2026 01:52
@github-actions github-actions Bot removed the CICD Run CI/CD label Apr 10, 2026
@rkishan516 rkishan516 added the CICD Run CI/CD label Apr 10, 2026
@rkishan516 rkishan516 force-pushed the simple-cross-imports-8 branch from e4c9a80 to 0ae6bea Compare April 10, 2026 14:22
@github-actions github-actions Bot removed the CICD Run CI/CD label Apr 10, 2026
@rkishan516 rkishan516 added the CICD Run CI/CD label Apr 10, 2026
@rkishan516 rkishan516 force-pushed the simple-cross-imports-8 branch from 0ae6bea to 858bfa8 Compare April 10, 2026 23:01
@github-actions github-actions Bot removed the CICD Run CI/CD label Apr 10, 2026
@rkishan516 rkishan516 added the CICD Run CI/CD label Apr 10, 2026
@rkishan516 rkishan516 force-pushed the simple-cross-imports-8 branch from 858bfa8 to 057fb3f Compare April 15, 2026 15:24
@github-actions github-actions Bot removed the CICD Run CI/CD label Apr 15, 2026
@rkishan516 rkishan516 added the CICD Run CI/CD label Apr 15, 2026
@rkishan516 rkishan516 force-pushed the simple-cross-imports-8 branch from 4fda411 to 4e79142 Compare April 24, 2026 06:37
@rkishan516 rkishan516 added the CICD Run CI/CD label Apr 24, 2026
@flutter-dashboard

Copy link
Copy Markdown

Golden file changes are available for triage from new commit, Click here to view.

For more guidance, visit Writing a golden file test for package:flutter.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

Changes reported for pull request #184798 at sha 4e79142

@rkishan516 rkishan516 requested a review from justinmc April 24, 2026 08:38

@justinmc justinmc 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.

LGTM 👍 . I approved all of the goldens, thanks! They were easy approvals now, the only change was TestCheckbox's slightly different look compared with Checkbox, and a very slight difference between TextField and TestTextField.

@justinmc justinmc added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 24, 2026
@auto-submit auto-submit Bot added this pull request to the merge queue Apr 24, 2026
Merged via the queue into flutter:master with commit 7578c75 Apr 24, 2026
167 of 168 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 24, 2026
@rkishan516 rkishan516 deleted the simple-cross-imports-8 branch May 16, 2026 06:15
pull Bot pushed a commit to chunfengyao/flutter that referenced this pull request May 16, 2026
This PR removes Material imports from:

- default_text_editing_shortcuts_test.dart
- framework_test.dart
- scrollable_in_overlay_test.dart

Also moved `TestRadio` & `TestSlider` widgets to their respective
separate files `radio_tester.dart` & `slider_tester.dart`

part of: flutter#177415 
dependent on flutter#184798 

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [AI contribution guidelines] and understand my
responsibilities, or I am not using AI tools.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [x] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [x] I followed the [breaking change policy] and added [Data Driven
Fixes] where supported.
- [x] All existing and new tests are passing.

---------

Co-authored-by: rkishan516 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) a: text input Entering text in a text field or keyboard related problems CICD Run CI/CD f: routes Navigator, Router, and related APIs. f: scrolling Viewports, list views, slivers, etc. framework flutter/packages/flutter repository. See also f: labels. will affect goldens Changes to golden files

Projects

Development

Successfully merging this pull request may close these issues.

3 participants