Skip to content

Remove deprecated RaisedButton - #98547

Merged
Piinks merged 4 commits into
flutter:masterfrom
Piinks:raisedButton
Apr 6, 2022
Merged

Piinks merged 4 commits into
flutter:masterfrom
Piinks:raisedButton

Conversation

@Piinks

@Piinks Piinks commented Feb 16, 2022 •

Copy link
Copy Markdown
Contributor

#98546 needs to land first ✅

This removes the deprecated RaisedButton widget.

This deprecated property has reached end of life after the release of Flutter 2.10

Part of #98537
For the full list of deprecations being removed in this batch, see flutter.dev/go/deprecations-removed-after-2-10

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 Piinks added c: contributor-productivity Team-specific productivity, code health, technical debt. framework flutter/packages/flutter repository. See also f: labels. c: API break Backwards-incompatible API changes a: quality A truly polished experience c: tech-debt Technical debt, code quality, testing, etc. labels Feb 16, 2022
@flutter-dashboard flutter-dashboard Bot added a: text input Entering text in a text field or keyboard related problems p: material_ui material_ui package in flutter/packages labels Feb 16, 2022
@Piinks

Piinks commented Mar 3, 2022

Copy link
Copy Markdown
Contributor Author

Still migrating customers here.

@flutter-dashboard

Copy link
Copy Markdown

This pull request executed golden file tests, but it has not been updated in a while (20+ days). Test results from Gold expire after as many days, so this pull request will need to be updated with a fresh commit in order to get results from Gold.

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.

@Piinks Piinks changed the title [WIP] Remove deprecated RaisedButton Remove deprecated RaisedButton Apr 1, 2022

@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 👍

Really cool to see -1300 lines!

@Piinks

Piinks commented Apr 6, 2022

Copy link
Copy Markdown
Contributor Author

The autosubmit bot does not appear to be working, merging manually.

@HansMuller

Copy link
Copy Markdown
Contributor

👍👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: quality A truly polished experience a: text input Entering text in a text field or keyboard related problems c: API break Backwards-incompatible API changes c: contributor-productivity Team-specific productivity, code health, technical debt. c: tech-debt Technical debt, code quality, testing, etc. framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants