Skip to content

Conversation

@TahaTesser
Copy link
Member

@TahaTesser TahaTesser commented May 17, 2023

fixes flutter/flutter#126920

TabBar.tabAlignment was introduced in flutter/flutter#125036

This PR adds a migration guide for TabBar.tabAlignment.

Related flutter/flutter#125974

Presubmit checklist

Copy link
Contributor

@Piinks Piinks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great! Just a couple of small nits

Copy link
Contributor

@atsansone atsansone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TahaTesser : A few changes for length and semantic breaks. Thanks!

@atsansone atsansone added the review.await-update Awaiting Updates after Edits label May 22, 2023
@TahaTesser
Copy link
Member Author

@atsansone
Appreciate the review, made the requested changes

@TahaTesser TahaTesser removed the review.await-update Awaiting Updates after Edits label May 22, 2023
@TahaTesser TahaTesser requested review from Piinks and atsansone May 22, 2023 12:22
@atsansone atsansone added the review.await-update Awaiting Updates after Edits label May 24, 2023

When you set `ThemeData.useMaterial3` to `true`,
a scrollable `TabBar` aligns tabs as `TabAlignment.startOffset` by default.
To change this alignment, set the `TabBar.tabAlignment` property.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can also use the theme, correct?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's right.

Copy link
Contributor

@atsansone atsansone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TahaTesser : I'll approve this as is. Keep in mind for next time that we hard wrap lines at 80 characters. Thanks!

@sfshaza2 sfshaza2 merged commit c419f60 into flutter:main May 26, 2023
@TahaTesser TahaTesser deleted the tab-alignment-migration branch May 26, 2023 20:28
@sfshaza2 sfshaza2 removed the review.await-update Awaiting Updates after Edits label Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create a migration guide for TabBar.TabAlignment

4 participants