Skip to content

Introduce Material 3 year2023 flag to SliderThemeData - #159721

Merged
TahaTesser merged 2 commits into
flutter:masterfrom
TahaTesser:year2023_slider_theme
Dec 6, 2024
Merged

TahaTesser merged 2 commits into
flutter:masterfrom
TahaTesser:year2023_slider_theme

Conversation

@TahaTesser

Copy link
Copy Markdown
Contributor

Related to Introduce Material 3 year2023 flag to the updated widget themes

Pre-launch Checklist

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

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages labels Dec 3, 2024
@TahaTesser
TahaTesser marked this pull request as ready for review December 3, 2024 14:52
@TahaTesser
TahaTesser requested a review from QuncCccccc December 3, 2024 14:52
Comment thread packages/flutter/lib/src/material/slider.dart Outdated
Comment thread packages/flutter/lib/src/material/slider.dart Outdated
Comment thread packages/flutter/lib/src/material/slider_theme.dart Outdated
Comment thread packages/flutter/lib/src/material/slider_theme.dart Outdated
Comment thread packages/flutter/test/material/slider_theme_test.dart Outdated
Comment thread packages/flutter/test/material/slider_theme_test.dart Outdated
Comment thread packages/flutter/test/material/slider_theme_test.dart Outdated

@QuncCccccc QuncCccccc 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 so much for the contribution! Looks good to me. Just left some nits in doc and comments in tests.

@TahaTesser
TahaTesser requested a review from QuncCccccc December 5, 2024 12:15

@QuncCccccc QuncCccccc 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 help! Looks like we also have some conflict to fix:)

Comment thread packages/flutter/test/material/slider_theme_test.dart Outdated
Comment thread packages/flutter/test/material/slider_theme_test.dart Outdated
@TahaTesser
TahaTesser enabled auto-merge December 6, 2024 14:45
@TahaTesser
TahaTesser added this pull request to the merge queue Dec 6, 2024
Merged via the queue into flutter:master with commit ae736e2 Dec 6, 2024
@TahaTesser
TahaTesser deleted the year2023_slider_theme branch December 6, 2024 14:47
@TahaTesser

TahaTesser commented Dec 6, 2024 •

Copy link
Copy Markdown
Contributor Author

This is weird. This PR is merged before all the tests are done when in merge queue. Shouldn't it wait for all tests to pass before merging?

Update:

Post about this behavior on Discord infra channel.

@Piinks

Piinks commented Dec 6, 2024

Copy link
Copy Markdown
Contributor

The autosubmit label should still be used. The button is still the same as if it were squash and merge - you are manually overriding presubmit checks and putting it in the merge queue.

@TahaTesser

Copy link
Copy Markdown
Contributor Author

The autosubmit label should still be used. The button is still the same as if it were squash and merge - you are manually overriding presubmit checks and putting it in the merge queue.

Thanks for the info. I will keep using the autosubmit label.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 6, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 9, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 10, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 11, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 11, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 11, 2024
@reidbaker reidbaker mentioned this pull request Dec 13, 2024
9 of 11 tasks
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 12, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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