Skip to content

Add a simplified SimpleCascadingMenuApp example - #149147

Merged
auto-submit[bot] merged 51 commits into
flutter:masterfrom
kaljitism:simple-cascading-menu-app
Jun 3, 2024
Merged

auto-submit[bot] merged 51 commits into
flutter:masterfrom
kaljitism:simple-cascading-menu-app

Conversation

@kaljitism

@kaljitism kaljitism commented May 27, 2024 •

Copy link
Copy Markdown
Contributor

The current MenuAnchor example in the API Docs is comprehensive and complicated for beginners. I have added a simple bare bone example without shortcuts, enums, etc in examples/api/lib/material/menu_anchor/ as menu_anchor.3.dart. The example is contributed by @mafreud

Fixes #148104

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 framework flutter/packages/flutter repository. See also f: labels. d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos labels May 27, 2024
@github-actions github-actions Bot added the p: material_ui material_ui package in flutter/packages label May 28, 2024
@TahaTesser
TahaTesser self-requested a review May 28, 2024 11:55

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

Thanks for the contribution!

Comment thread examples/api/lib/material/menu_anchor/menu_anchor.3.dart Outdated
Comment thread examples/api/lib/material/menu_anchor/menu_anchor.3.dart Outdated
Comment thread examples/api/lib/material/menu_anchor/menu_anchor.3.dart Outdated
Comment thread examples/api/lib/material/menu_anchor/menu_anchor.3.dart Outdated
Comment thread examples/api/lib/material/menu_anchor/menu_anchor.3.dart Outdated
Comment thread examples/api/lib/material/menu_anchor/menu_anchor.3.dart Outdated
Comment thread examples/api/lib/material/menu_anchor/menu_anchor.3.dart Outdated
Comment thread examples/api/lib/material/menu_anchor/menu_anchor.3.dart Outdated
Comment thread packages/flutter/lib/src/material/menu_anchor.dart Outdated
Comment thread packages/flutter/lib/src/material/menu_anchor.dart Outdated
@TahaTesser TahaTesser changed the title added the SimpleCascadingMenuApp example. Add a simplified SimpleCascadingMenuApp example May 29, 2024
kaljitism and others added 2 commits June 2, 2024 14:17
@kaljitism

kaljitism commented Jun 2, 2024 •

Copy link
Copy Markdown
Contributor Author

Sorry for late update, I had my monthly distro change ceremony!

@TahaTesser @bleroux If there are any other such issues like these - Missing examples/ tests/ simple bugs, you could assign me, I can work on. few issues daily and reduce the friction of the team!

@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 #149147 at sha ec25f74

@kaljitism
kaljitism requested a review from bleroux June 3, 2024 05:09

@bleroux bleroux 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! Thanks for the contribution.

@bleroux

bleroux commented Jun 3, 2024

Copy link
Copy Markdown
Contributor

@kaljitism Can you try to rebase this PR? It might help with the flutter-gold step which is wrongly complaining about a golden change.

@kaljitism

Copy link
Copy Markdown
Contributor Author

@bleroux Why I am failing the tree-status test? How can I fix this?

@TahaTesser TahaTesser added the autosubmit Merge PR when tree becomes green via auto submit App label Jun 3, 2024
@auto-submit
auto-submit Bot merged commit 0e7295f into flutter:master Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 3, 2024
hello-coder-xu pushed a commit to hello-coder-xu/flutter that referenced this pull request Jun 4, 2024
…pdate

* master: (168 commits)
  Fix the scrolling layout deviation of `CupertinoActionSheet` (flutter#149439)
  Roll Flutter Engine from 60a7bb2 to a6aa5d8 (2 revisions) (flutter#149627)
  Roll Flutter Engine from ea72558 to 60a7bb2 (2 revisions) (flutter#149623)
  Place `flutter_gpu` in the package cache. (flutter#149299)
  Switch to triage-* labels for platform package triage (flutter#149614)
  Roll pub packages (flutter#149617)
  Fixes multi line textfield hint text gets ellipsized (flutter#148423)
  Support failures-only and silent reporters in `flutter test` (flutter#148739)
  [CupertinoActionSheet] Fix overflow of the overscroll section when the user scrolls far (flutter#149542)
  Fix InputDecorator.prefixIcon color when disabled (flutter#149595)
  Added filter callback on dropdown menu (flutter#143939)
  update generated localized message files in the stocks test app (flutter#148741)
  Add a simplified SimpleCascadingMenuApp example (flutter#149147)
  Reland "Prevent LayoutBuilder from rebuilding more than once (flutter#147856)" (flutter#149303)
  Move some benchmarks from MotoG4 to Mokey (flutter#149567)
  Roll Packages from d8e8e8c to 11e192a (2 revisions) (flutter#149596)
  Cleanup triage reports from docs/ (flutter#149545)
  Roll Flutter Engine from d81edf6 to ea72558 (1 revision) (flutter#149590)
  Roll Flutter Engine from b0f4d74 to d81edf6 (1 revision) (flutter#149468)
  Roll Flutter Engine from 40b868e to b0f4d74 (1 revision) (flutter#149467)
  ...
hello-coder-xu added a commit to hello-coder-xu/flutter that referenced this pull request Jun 6, 2024
…pdate

* master: (168 commits)
  Fix the scrolling layout deviation of `CupertinoActionSheet` (flutter#149439)
  Roll Flutter Engine from 60a7bb2 to a6aa5d8 (2 revisions) (flutter#149627)
  Roll Flutter Engine from ea72558 to 60a7bb2 (2 revisions) (flutter#149623)
  Place `flutter_gpu` in the package cache. (flutter#149299)
  Switch to triage-* labels for platform package triage (flutter#149614)
  Roll pub packages (flutter#149617)
  Fixes multi line textfield hint text gets ellipsized (flutter#148423)
  Support failures-only and silent reporters in `flutter test` (flutter#148739)
  [CupertinoActionSheet] Fix overflow of the overscroll section when the user scrolls far (flutter#149542)
  Fix InputDecorator.prefixIcon color when disabled (flutter#149595)
  Added filter callback on dropdown menu (flutter#143939)
  update generated localized message files in the stocks test app (flutter#148741)
  Add a simplified SimpleCascadingMenuApp example (flutter#149147)
  Reland "Prevent LayoutBuilder from rebuilding more than once (flutter#147856)" (flutter#149303)
  Move some benchmarks from MotoG4 to Mokey (flutter#149567)
  Roll Packages from d8e8e8c to 11e192a (2 revisions) (flutter#149596)
  Cleanup triage reports from docs/ (flutter#149545)
  Roll Flutter Engine from d81edf6 to ea72558 (1 revision) (flutter#149590)
  Roll Flutter Engine from b0f4d74 to d81edf6 (1 revision) (flutter#149468)
  Roll Flutter Engine from 40b868e to b0f4d74 (1 revision) (flutter#149467)
  ...
hello-coder-xu added a commit to hello-coder-xu/flutter that referenced this pull request Jun 9, 2024
…pdate

* master: (181 commits)
  Fix the scrolling layout deviation of `CupertinoActionSheet` (flutter#149439)
  Roll Flutter Engine from 60a7bb2 to a6aa5d8 (2 revisions) (flutter#149627)
  Roll Flutter Engine from ea72558 to 60a7bb2 (2 revisions) (flutter#149623)
  Place `flutter_gpu` in the package cache. (flutter#149299)
  Switch to triage-* labels for platform package triage (flutter#149614)
  Roll pub packages (flutter#149617)
  Fixes multi line textfield hint text gets ellipsized (flutter#148423)
  Support failures-only and silent reporters in `flutter test` (flutter#148739)
  [CupertinoActionSheet] Fix overflow of the overscroll section when the user scrolls far (flutter#149542)
  Fix InputDecorator.prefixIcon color when disabled (flutter#149595)
  Added filter callback on dropdown menu (flutter#143939)
  update generated localized message files in the stocks test app (flutter#148741)
  Add a simplified SimpleCascadingMenuApp example (flutter#149147)
  Reland "Prevent LayoutBuilder from rebuilding more than once (flutter#147856)" (flutter#149303)
  Move some benchmarks from MotoG4 to Mokey (flutter#149567)
  Roll Packages from d8e8e8c to 11e192a (2 revisions) (flutter#149596)
  Cleanup triage reports from docs/ (flutter#149545)
  Roll Flutter Engine from d81edf6 to ea72558 (1 revision) (flutter#149590)
  Roll Flutter Engine from b0f4d74 to d81edf6 (1 revision) (flutter#149468)
  Roll Flutter Engine from 40b868e to b0f4d74 (1 revision) (flutter#149467)
  ...
hello-coder-xu added a commit to hello-coder-xu/flutter that referenced this pull request Jun 11, 2024
* master: (181 commits)
  Fix the scrolling layout deviation of `CupertinoActionSheet` (flutter#149439)
  Roll Flutter Engine from 60a7bb2 to a6aa5d8 (2 revisions) (flutter#149627)
  Roll Flutter Engine from ea72558 to 60a7bb2 (2 revisions) (flutter#149623)
  Place `flutter_gpu` in the package cache. (flutter#149299)
  Switch to triage-* labels for platform package triage (flutter#149614)
  Roll pub packages (flutter#149617)
  Fixes multi line textfield hint text gets ellipsized (flutter#148423)
  Support failures-only and silent reporters in `flutter test` (flutter#148739)
  [CupertinoActionSheet] Fix overflow of the overscroll section when the user scrolls far (flutter#149542)
  Fix InputDecorator.prefixIcon color when disabled (flutter#149595)
  Added filter callback on dropdown menu (flutter#143939)
  update generated localized message files in the stocks test app (flutter#148741)
  Add a simplified SimpleCascadingMenuApp example (flutter#149147)
  Reland "Prevent LayoutBuilder from rebuilding more than once (flutter#147856)" (flutter#149303)
  Move some benchmarks from MotoG4 to Mokey (flutter#149567)
  Roll Packages from d8e8e8c to 11e192a (2 revisions) (flutter#149596)
  Cleanup triage reports from docs/ (flutter#149545)
  Roll Flutter Engine from d81edf6 to ea72558 (1 revision) (flutter#149590)
  Roll Flutter Engine from b0f4d74 to d81edf6 (1 revision) (flutter#149468)
  Roll Flutter Engine from 40b868e to b0f4d74 (1 revision) (flutter#149467)
  ...
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 6, 2024
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 d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages will affect goldens Changes to golden files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MenuAnchor] needs simpler example

3 participants