Fix `DropdownButton` menu clip #102970

TahaTesser · 2022-05-03T09:35:14Z

minimal code sample

import 'package:flutter/material.dart';

void main() {
  runApp(const App());
}

class App extends StatelessWidget {
  const App({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        brightness: Brightness.light,
      ),
      home: const Home(),
    );
  }
}

class Home extends StatefulWidget {
  const Home({Key? key}) : super(key: key);

  @override
  State<Home> createState() => _HomeState();
}

class _HomeState extends State<Home> {
  int _value = 0;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.blue,

      body: Padding(
        padding: const EdgeInsets.symmetric(
          vertical: 8.0,
          horizontal: 16.0,
        ),
        child: MyDropdown(
          value: _value,
          onChanged: (value) {
            if (value != null && value != _value) {
              setState(() {
                _value = value;
              });
            }
          },
        ),
      ),
    );
  }
}

class MyDropdown extends StatelessWidget {
  const MyDropdown({
    Key? key,
    required this.value,
    required this.onChanged,
  }) : super(key: key);

  final int value;
  final ValueChanged<int?> onChanged;

  @override
  Widget build(BuildContext context) {
    return DropdownButton<int>(
      isExpanded: true,
      onChanged: onChanged,
      value: value,
      borderRadius: const BorderRadius.all(
        Radius.circular(32.0),
      ),
      dropdownColor: Colors.red[700],
      items: [
        for (int i = 0; i < 100; i++)
          DropdownMenuItem(
            value: i,
            child: Text(
              i.toString(),
            ),
          ),
      ],
    );
  }
}

Bug

Fixed

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.
All existing and new tests are passing.

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

Piinks

Nice fix! Thank you! LGTM

fluttergithubbot · 2022-05-06T23:19:11Z

This pull request is not suitable for automatic merging in its current state.

This commit is not mergeable and has conflicts. Please rebase your PR and fix all the conflicts.

Roll Flutter from b305eb6 to 4c57320 (33 revisions) flutter/flutter@b305eb6...4c57320 2022-05-09 engine-flutter-autoroll@skia.org Roll Flutter Engine from 71b9adccfb63 to 6248a5a9ae28 (1 revision) (flutter/flutter#103350) 2022-05-09 engine-flutter-autoroll@skia.org Roll Flutter Engine from 3af7fb24d4dd to 71b9adccfb63 (1 revision) (flutter/flutter#103347) 2022-05-09 jesus_sguerrero@hotmail.com [flutter_tools] stringArg refactor (flutter/flutter#103231) 2022-05-09 maistho@gmail.com Replace ListView with ListView.builder for LicensesPage (flutter/flutter#102692) 2022-05-09 gspencergoog@users.noreply.github.com Adds `menuBarMenuLabel`, and removes unneeded key localizations (flutter/flutter#102100) 2022-05-09 engine-flutter-autoroll@skia.org Roll Plugins from 49b06b3 to c5c319a (15 revisions) (flutter/flutter#103345) 2022-05-09 engine-flutter-autoroll@skia.org Roll Engine from e7c44a582c58 to 3af7fb24d4dd (1 revision) (flutter/flutter#103334) 2022-05-09 engine-flutter-autoroll@skia.org Roll Engine from c02712fe1160 to e7c44a582c58 (1 revision) (flutter/flutter#103333) 2022-05-09 engine-flutter-autoroll@skia.org Roll Engine from 6efd8f00dcc6 to c02712fe1160 (1 revision) (flutter/flutter#103327) 2022-05-09 tessertaha@gmail.com `CupertinoTabScaffold`/`CupertinoTabController`: Add interactive examples (flutter/flutter#103196) 2022-05-09 engine-flutter-autoroll@skia.org Roll Engine from c930e64a13bc to 6efd8f00dcc6 (1 revision) (flutter/flutter#103323) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from d0e6e14d27cc to c930e64a13bc (2 revisions) (flutter/flutter#103308) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from fffb3eb58e7d to d0e6e14d27cc (1 revision) (flutter/flutter#103307) 2022-05-08 danny@tuppeny.com Use consistent date instead of DateTime.now() in evaluation tests (flutter/flutter#103269) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from 259ccadd8203 to fffb3eb58e7d (1 revision) (flutter/flutter#103298) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from 0b84783081a7 to 259ccadd8203 (1 revision) (flutter/flutter#103293) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from 19d5f037f221 to 0b84783081a7 (1 revision) (flutter/flutter#103291) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from 2d1f34cbdde2 to 19d5f037f221 (1 revision) (flutter/flutter#103289) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from b89e093d887f to 2d1f34cbdde2 (1 revision) (flutter/flutter#103287) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from aa80520b7fc9 to b89e093d887f (1 revision) (flutter/flutter#103286) 2022-05-08 engine-flutter-autoroll@skia.org Roll Engine from 4c05cb763866 to aa80520b7fc9 (1 revision) (flutter/flutter#103283) 2022-05-07 engine-flutter-autoroll@skia.org Roll Engine from 235d951ccfbd to 4c05cb763866 (1 revision) (flutter/flutter#103279) 2022-05-07 engine-flutter-autoroll@skia.org Roll Engine from 24466845e7a9 to 235d951ccfbd (4 revisions) (flutter/flutter#103276) 2022-05-07 engine-flutter-autoroll@skia.org Roll Engine from 2d2f8629cdb2 to 24466845e7a9 (1 revision) (flutter/flutter#103274) 2022-05-07 engine-flutter-autoroll@skia.org Roll Engine from 4a358030cb07 to 2d2f8629cdb2 (1 revision) (flutter/flutter#103273) 2022-05-07 fujino@google.com [flutter_tools] add --uninstall-first flag and pipe it through to ios-deploy (flutter/flutter#102948) 2022-05-07 xubaolin@oppo.com Fix a `_WrappedScrollBehavior.shouldNotify` bug (flutter/flutter#103267) 2022-05-07 magder@google.com Remove custom `unawaited`, prefer `dart:async` version (flutter/flutter#103212) 2022-05-07 36861262+QuncCccccc@users.noreply.github.com Added clipBehavior on TabBarView (flutter/flutter#103166) 2022-05-07 leroux_bruno@yahoo.fr Fix empty Stack with infinite constraints throws (flutter/flutter#102642) 2022-05-07 42252197+egramond@users.noreply.github.com Updating the Slider Widget to allow up and down arrow keys to navigate out of the slider when in directional NavigationMode. (flutter/flutter#103149) 2022-05-07 tessertaha@gmail.com Fix `DropdownButton` menu clip (flutter/flutter#102970) 2022-05-07 engine-flutter-autoroll@skia.org Roll Engine from 094cb30943a5 to 4a358030cb07 (1 revision) (flutter/flutter#103260) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC stuartmorgan@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: ...

This reverts commit 2f65753.

Piinks · 2022-05-19T18:36:42Z

packages/flutter/lib/src/material/dropdown.dart

-                    shrinkWrap: true,
-                    children: children,
+          child: ClipRRect(
+            borderRadius: widget.borderRadius ?? BorderRadius.zero,


Since this is being reverted, it may be that we only want to apply this clip when the border radius is provided by the user. Otherwise the clipBehavior can be Clip.none.

flutter-dashboard bot added f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. labels May 3, 2022

TahaTesser requested a review from Piinks May 3, 2022 09:35

Piinks approved these changes May 6, 2022

View reviewed changes

Piinks added the waiting for tree to go green label May 6, 2022

fluttergithubbot removed the waiting for tree to go green label May 6, 2022

Fix DropdownButton menu clip

5d3184e

TahaTesser force-pushed the fix_dropdown_menu_clip branch from 6666e3b to 5d3184e Compare May 7, 2022 07:35

TahaTesser added the waiting for tree to go green label May 7, 2022

fluttergithubbot merged commit 2f65753 into flutter:master May 7, 2022

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from b305eb633780 to 2f657536c883 (2 revisions) flutter/packages#1813

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

d0b15f1

TahaTesser deleted the fix_dropdown_menu_clip branch May 7, 2022 09:58

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from 90868d3ba33a to ae7fcc7e51f6 (2 revisions) flutter/plugins#5663

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

b2864a0

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from b305eb633780 to ae7fcc7e51f6 (3 revisions) flutter/packages#1814

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

6c2a247

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from b305eb633780 to c18097178c91 (4 revisions) flutter/packages#1815

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

b71ca21

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from b305eb633780 to c18097178c91 (4 revisions) flutter/packages#1816

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

3a74585

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from b305eb633780 to 2427d4f3d630 (5 revisions) flutter/packages#1817

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

41c5342

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from b305eb633780 to c6ced845e389 (6 revisions) flutter/packages#1818

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

8c17634

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from b305eb633780 to c6ced845e389 (6 revisions) flutter/packages#1819

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

8c1a803

engine-flutter-autoroll mentioned this pull request May 7, 2022

Roll Flutter from b305eb633780 to 4bed76757db7 (7 revisions) flutter/packages#1820

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 7, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

bfc5948

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

77c8459

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 84ffdad368d9 (28 revisions) flutter/packages#1848

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

ebce509

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 6504f2896ccb (29 revisions) flutter/packages#1849

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

fee796f

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 95116386b73b (30 revisions) flutter/packages#1850

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

54ee91e

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 596e9d1c47ac (31 revisions) flutter/packages#1851

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

02303f5

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 65caffd40537 (32 revisions) flutter/packages#1853

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

aa3ee12

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 4c573207acc3 (33 revisions) flutter/packages#1854

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

41cc40d

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 106ab98f878b (34 revisions) flutter/packages#1855

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

044a4b0

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 89f755c2df3e (35 revisions) flutter/packages#1856

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

11a4977

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 0ff0affb3a9d (36 revisions) flutter/packages#1858

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

a0813ac

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to ff136cba7fe0 (37 revisions) flutter/packages#1859

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

e944102

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 16f3feaf0780 (38 revisions) flutter/packages#1860

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

23c262c

engine-flutter-autoroll mentioned this pull request May 9, 2022

Roll Flutter from b305eb633780 to 93cce92ea827 (39 revisions) flutter/packages#1861

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 9, 2022

2f65753 Fix DropdownButton menu clip (flutter/flutter#102970)

f6555ae

Piinks added a commit that referenced this pull request May 19, 2022

Revert "Fix DropdownButton menu clip (#102970)"

d6e000d

This reverts commit 2f65753.

Piinks mentioned this pull request May 19, 2022

Revert "Fix DropdownButton menu clip" #104203

Merged

Piinks reviewed May 19, 2022

View reviewed changes

TahaTesser mentioned this pull request May 20, 2022

[Reland]: Fix DropdownButton menu clip #104251

Merged

8 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fix `DropdownButton` menu clip #102970

Fix `DropdownButton` menu clip #102970

Uh oh!

TahaTesser commented May 3, 2022 •

edited

Loading

Uh oh!

Piinks left a comment

Uh oh!

fluttergithubbot commented May 6, 2022

Uh oh!

Piinks May 19, 2022

Uh oh!

TahaTesser May 19, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Fix DropdownButton menu clip #102970

Fix DropdownButton menu clip #102970

Uh oh!

Conversation

TahaTesser commented May 3, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Bug

Fixed

Pre-launch Checklist

Uh oh!

Piinks left a comment

Choose a reason for hiding this comment

Uh oh!

fluttergithubbot commented May 6, 2022

Uh oh!

Piinks May 19, 2022

Choose a reason for hiding this comment

Uh oh!

TahaTesser May 19, 2022

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Fix `DropdownButton` menu clip #102970

Fix `DropdownButton` menu clip #102970

TahaTesser commented May 3, 2022 •

edited

Loading