packages icon indicating copy to clipboard operation
packages copied to clipboard

[flutter_adaptive_scaffold] Surface `useLegacyColorScheme` property in `AdaptiveScaffold`

Open a-wallen opened this issue 1 year ago • 3 comments

Surfaces the useLegacyColorScheme property from BottomNavigationBar.

Fixes https://github.com/flutter/flutter/issues/116903

Pre-launch Checklist

  • [x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • [x] I read the Tree Hygiene wiki page, which explains my responsibilities.
  • [x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use dart format.)
  • [x] I signed the CLA.
  • [x] The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • [x] I listed at least one issue that this PR fixes in the description above.
  • [x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
  • [x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
  • [x] 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.

a-wallen avatar Jan 05 '23 20:01 a-wallen

Closing as this has been draft for a while; please feel free to re-open if this is something you plan to revisit.

stuartmorgan avatar Feb 14 '23 20:02 stuartmorgan

@stuartmorgan no problem. FWIW this PR can land once useLegacyColorScheme rolls into stable, but since that property hasn't rolled on that branch I marked the PR as a draft. I'm not sure if that changes your decision about closing that PR.

a-wallen avatar Feb 17 '23 16:02 a-wallen

Sure, if it's good to go other than that, leaving it open makes sense. Thanks!

stuartmorgan avatar Feb 17 '23 18:02 stuartmorgan

auto label is removed for flutter/packages, pr: 3025, due to This PR has not met approval requirements for merging. You have project association CONTRIBUTOR and need 1 more review(s) in order to merge this PR.

  • Merge guidelines: You need at least one approved review if you are already a MEMBER or two member reviews if you are not a MEMBER before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

auto-submit[bot] avatar May 02 '23 22:05 auto-submit[bot]

auto label is removed for flutter/packages, pr: 3025, due to Validations Fail.

auto-submit[bot] avatar May 02 '23 22:05 auto-submit[bot]

Oh, I guess it's not defined in 3.3.10 (it is in the stable 3.6.0 branch in my Flutter, though).

I guess we'll wait until the legacy version catches up.

gspencergoog avatar May 02 '23 23:05 gspencergoog

SGTM. I guess I'll be seeing you again when that happens @gspencergoog.

a-wallen avatar May 02 '23 23:05 a-wallen

Syncing with main now that everything has rolled forward for the 3.10 release.

stuartmorgan avatar May 16 '23 19:05 stuartmorgan

I pinged @Hixie in Discord about the CLA problem.

stuartmorgan avatar May 16 '23 19:05 stuartmorgan

The CLA issue is fixed, but we'll need to update this to resolve the conflicts. CHANGELOG is the usual trivial change, but there's a code conflict too.

stuartmorgan avatar May 23 '23 19:05 stuartmorgan

@gspencergoog Do you have time to fix the conflicts here, or should I add that to my list?

stuartmorgan avatar Jun 06 '23 19:06 stuartmorgan

I could fix the conflicts, but NavigationBar doesn't even have useLegacyColorScheme, so I'm thinking that we could just close this PR, since we're not using BottomNavigationBar anymore.

gspencergoog avatar Jun 06 '23 20:06 gspencergoog