Skip to content

[material_ui] Add scrollPadding property to DropdownMenuFormField - #12735

Merged
auto-submit[bot] merged 6 commits into
flutter:mainfrom
Hamidrzash:add-scroll-padding-to-dropdown-form-field
Oct 5, 2026
Merged

auto-submit[bot] merged 6 commits into
flutter:mainfrom
Hamidrzash:add-scroll-padding-to-dropdown-form-field

Conversation

@Hamidrzash

@Hamidrzash Hamidrzash commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor

Port of flutter/flutter#184059 to flutter/packages, per the instructions in flutter/flutter#188444.

This change adds the scrollPadding property to DropdownMenuFormField and forwards it to the underlying DropdownMenu, bringing the form field variant into parity with DropdownMenu.

Related Issues

The original issue has been closed, but the following comment explains the remaining gap that this PR addresses:

flutter/flutter#182890 (comment)

The merged PR #183109 covers DropdownMenu, but DropdownMenuFormField still doesn't expose scrollPadding. I'm opening a follow-up PR to address that remaining gap. #184059

Pre-Review Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@QuncCccccc
QuncCccccc requested review from QuncCccccc and elliette and removed request for QuncCccccc and elliette September 8, 2026 21:31
@QuncCccccc

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Because of the volume of PRs we receive, we require that new contributors use our checklist to guide them through critical steps in creating a Flutter PR. This PR's description is missing that checklist, so it is being marked as a Draft.

Please edit the PR description to add the checklist, then ensure that you have completed all of the steps. Once you've done that, please mark the PR as ready for review.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@QuncCccccc
QuncCccccc marked this pull request as draft September 8, 2026 21:32
@Hamidrzash
Hamidrzash marked this pull request as ready for review September 8, 2026 21:51

@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 introduces the scrollPadding property to DropdownMenuFormField, forwarding it to the underlying DropdownMenu. A widget test has been added to verify that the property is correctly passed, and a pending changelog entry has been created. There are no review comments, and I have no feedback to provide.

@Hamidrzash

Copy link
Copy Markdown
Contributor Author

@QuncCccccc Thanks.. I've added the required checklist and marked the PR as ready for review.

@navaronbracke navaronbracke 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!

@navaronbracke navaronbracke added the CICD Run CI/CD label Sep 11, 2026
@flutter-dashboard flutter-dashboard Bot removed the CICD Run CI/CD label Sep 14, 2026
@dkwingsmt dkwingsmt added the CICD Run CI/CD label Sep 15, 2026
@flutter-dashboard flutter-dashboard Bot removed the CICD Run CI/CD label Sep 18, 2026
@dkwingsmt dkwingsmt added the CICD Run CI/CD label Sep 24, 2026

@dkwingsmt dkwingsmt 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. Thank you!

@Hamidrzash

Copy link
Copy Markdown
Contributor Author

Hi! Just checking in on this PR. Is there anything else needed from my side before merging?
Thanks!

@navaronbracke navaronbracke added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 5, 2026
@auto-submit
auto-submit Bot merged commit 951f2f7 into flutter:main Oct 5, 2026
14 checks passed
ZhuJHua pushed a commit to ZhuJHua/flutter that referenced this pull request Oct 6, 2026
…er#193840)

flutter/packages@5620e65...951f2f7

2026-10-05 [email protected] [material_ui] Add scrollPadding
property to DropdownMenuFormField (flutter/packages#12735)
2026-10-02 [email protected] [material_ui]
Migrate M3 Motion template to use new gen_defaults
(flutter/packages#13058)
2026-10-02 [email protected]
[cupertino_ui] Forward showDragHandle from showCupertinoSheet to
CupertinoSheetRoute (flutter/packages#13011)
2026-10-02 [email protected] [cupertino_ui]
Fix CupertinoMagnifier focal point with custom size
(flutter/packages#13029)
2026-10-02 [email protected] [ci] Add material_ui and cupertino_ui
to customer testing (flutter/packages#13114)
2026-10-02 [email protected] [material_ui]
Add Material 3 Expressive migration skill (flutter/packages#12818)
2026-10-02 [email protected] [video_player] Ignore late position
updates after disposal (flutter/packages#12971)
2026-10-02 [email protected] [material_ui]
Remove unconditional dart:io import from about.dart
(flutter/packages#12774)
2026-10-02 [email protected] [material_ui] Fix
DropdownButtonFormField underline alignment at bottom
(flutter/packages#12606)
2026-10-02 [email protected] Roll Flutter from
e89fd0a to d03768e (23 revisions) (flutter/packages#13111)
2026-10-02 [email protected] [material_ui] Migrate
`Switch` API doc snippets to {@example} (batch 8)
(flutter/packages#13052)
2026-10-02 [email protected]
[url_launcher] Fix supportsCloseForLaunchMode to query close support
(flutter/packages#12926)
2026-10-02 [email protected] Redistribute
iOS/macOS Suggested Reviewers (flutter/packages#13076)
2026-10-02 [email protected] [image_picker] Subsample large images
when resizing on Android (flutter/packages#13038)
2026-10-02 [email protected] [cupertino_ui] Remove
redundant null arguments in Completer.complete (flutter/packages#13008)
2026-10-02 [email protected] [repo]
Update AGENTS.md with dependency allowance and versioning guidance
(flutter/packages#12579)

If this roll has caused a breakage, revert this CL and set the roller
to dry run mode using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected] on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App CICD Run CI/CD p: material_ui triage-design Should be looked at in design triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants