Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: flutter/flutter
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 5d3bca4500b1
Choose a base ref
...
head repository: flutter/flutter
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 2e806700b928
Choose a head ref
  • 20 commits
  • 35 files changed
  • 8 contributors

Commits on Apr 25, 2024

  1. Configuration menu
    Copy the full SHA
    3033007 View commit details
    Browse the repository at this point in the history
  2. Roll Packages from cf6d280 to fde908d (11 revisions) (#147375)

    flutter/packages@cf6d280...fde908d
    
    2024-04-25 49699333+dependabot[bot]@users.noreply.github.com Bump actions/checkout from 4.1.3 to 4.1.4 (flutter/packages#6609)
    2024-04-24 [email protected] [go_router] Add `GoRouterState state` parameter to `GoRouterData.onExit` (flutter/packages#6495)
    2024-04-24 [email protected] Add CI steps to test iOS and macOS plugins with both CocoaPods and Swift Package Manager (flutter/packages#6557)
    2024-04-24 [email protected] Roll Flutter from 77043ba to dba4f77 (30 revisions) (flutter/packages#6607)
    2024-04-24 [email protected] [camera] Finish converting iOS to Pigeon (flutter/packages#6601)
    2024-04-24 [email protected] [go_router] Fixes an issue where route future does not complete when â�¦ (flutter/packages#6596)
    2024-04-24 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Bump legacy all_packages project AGP version to 7.0.0, Gradle version to 7.0.2 (#6591)" (flutter/packages#6605)
    2024-04-23 [email protected] [in_app_purchase_android] Readme update for Alternative billing (flutter/packages#6578)
    2024-04-23 [email protected] Bump legacy all_packages project AGP version to 7.0.0, Gradle version to 7.0.2 (flutter/packages#6591)
    2024-04-23 [email protected] Roll Flutter from 140edb9 to 77043ba (21 revisions) (flutter/packages#6599)
    2024-04-23 49699333+dependabot[bot]@users.noreply.github.com Bump github/codeql-action from 3.25.1 to 3.25.2 (flutter/packages#6597)
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
    Please CC [email protected],[email protected] on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
    
    To report a problem with the AutoRoller itself, please file a bug:
    https://issues.skia.org/issues/new?component=1389291&template=1850622
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
    engine-flutter-autoroll authored Apr 25, 2024
    Configuration menu
    Copy the full SHA
    439c03f View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    c855710 View commit details
    Browse the repository at this point in the history
  4. Makes badge to auto size with content (#146853)

    Makes badge sizes it self according to child.
    
    Previously, the bubble fixed its height to 16pixel. It clip the content
    if it is taller than 16 pixel. This causes an issue where user can
    increase font size in the android setting and mess up the ui
    
    Now, the bubble can have various height from 16 pixel to the height of
    the child, it also extend the width to be the same as height if the
    child's width < height.
    
    There is minor changes to floating location in the test. I can't really
    fix them unless I change some of the public API like `Badge(offset)` or
    `BadgeThemeData.largeSize`. They are used differently in the new logic
    since now the badge can be larger than the Badge.child. I figure they
    are kind of minor so i think it should be ok
    
    Before the fix
    
    ![image](https://github.com/flutter/flutter/assets/47866232/10430cb7-513a-4425-99ee-163c494abff8)
    
    
    After the fix
    
    ![image](https://github.com/flutter/flutter/assets/47866232/34390763-2778-4c4e-b468-eda972f61bb1)
    
    fixes #146777
    
    
    ## 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.
    
    If you need help, consider asking for advice on the #hackers-new channel
    on [Discord].
    
    <!-- Links -->
    [Contributor Guide]:
    https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
    [Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
    [test-exempt]:
    https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
    [Flutter Style Guide]:
    https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
    [Features we expect every widget to implement]:
    https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#features-we-expect-every-widget-to-implement
    [CLA]: https://cla.developers.google.com/
    [flutter/tests]: https://github.com/flutter/tests
    [breaking change policy]:
    https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
    [Discord]: https://github.com/flutter/flutter/wiki/Chat
    [Data Driven Fixes]:
    https://github.com/flutter/flutter/wiki/Data-driven-Fixes
    chunhtai authored Apr 25, 2024
    Configuration menu
    Copy the full SHA
    9c33723 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    0c231a4 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    96d9cd1 View commit details
    Browse the repository at this point in the history
  7. Roll Flutter Engine from 694756b875bf to 10a3504a9261 (3 revisions) (#…

    …147380)
    
    flutter/engine@694756b...10a3504
    
    2024-04-25 [email protected] Roll Dart SDK from b5f51d886819 to f9c9894a2500 (1 revision) (flutter/engine#52393)
    2024-04-25 [email protected] Roll Skia from b440433e2df5 to 67bba57bfe80 (1 revision) (flutter/engine#52391)
    2024-04-25 [email protected] Remove goma support from impeller-cmake-example (flutter/engine#52390)
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
    Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
    
    To report a problem with the AutoRoller itself, please file a bug:
    https://issues.skia.org/issues/new?component=1389291&template=1850622
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
    engine-flutter-autoroll authored Apr 25, 2024
    Configuration menu
    Copy the full SHA
    350ccec View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    ab715b1 View commit details
    Browse the repository at this point in the history
  9. [devicelab] explicitly enable vulkan validation in test. (#147382)

    Part of #142659
    
    this test expects validation layers on, so we need to explicitly enable
    them before we can turn them off by default.
    Jonah Williams authored Apr 25, 2024
    Configuration menu
    Copy the full SHA
    4e16e67 View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    014cf33 View commit details
    Browse the repository at this point in the history

Commits on Apr 26, 2024

  1. Configuration menu
    Copy the full SHA
    101ff6f View commit details
    Browse the repository at this point in the history
  2. add a new PopScope.onPopWithResultInvoke widget to replace PopScope.o…

    …nPopInvoke (#147016)
    
    same as #139164, but make it a soft transition
    chunhtai authored Apr 26, 2024
    Configuration menu
    Copy the full SHA
    8031a3e View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    c77fe6b View commit details
    Browse the repository at this point in the history
  4. Roll Flutter Engine from a09295fe03c0 to e99fc6ef91ef (4 revisions) (#…

    …147391)
    
    flutter/engine@a09295f...e99fc6e
    
    2024-04-25 [email protected] [Impeller] while we still have benchmarks, see if we're efficient enough for this to be faster. (flutter/engine#52398)
    2024-04-25 [email protected] [Impeller] Vulkan validation off by default. (flutter/engine#52397)
    2024-04-25 [email protected] Roll Skia from cb32ad619678 to 52083c205016 (1 revision) (flutter/engine#52396)
    2024-04-25 [email protected] [Impeller] kick off registration and initial PSO compilation of runtime effect earlier. (flutter/engine#52381)
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
    Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
    
    To report a problem with the AutoRoller itself, please file a bug:
    https://issues.skia.org/issues/new?component=1389291&template=1850622
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
    engine-flutter-autoroll authored Apr 26, 2024
    Configuration menu
    Copy the full SHA
    6d8639d View commit details
    Browse the repository at this point in the history
  5. Fix DateRangePickerDialog does not use `rangePickerHeaderBackground…

    …Color` from `DatePickerTheme` in M2 (#147370)
    
    fixes [DatePickerThemeData's `rangePickerHeaderBackgroundColor` not being applied to DateRangePickerDialog on M2](#147301)
    
    ### Code sample
    
    <details>
    <summary>expand to view the code sample</summary> 
    
    ```dart
    import 'package:flutter/material.dart';
    
    void main() => runApp(const MyApp());
    
    class MyApp extends StatelessWidget {
      const MyApp({super.key});
    
      @OverRide
      Widget build(BuildContext context) {
        return MaterialApp(
          debugShowCheckedModeBanner: false,
          theme: ThemeData(
            useMaterial3: false,
            datePickerTheme: const DatePickerThemeData(
              rangePickerHeaderBackgroundColor: Colors.red,
            ),
          ),
          home: Scaffold(
            body: Center(
              child: Builder(builder: (BuildContext context) {
                return ElevatedButton(
                  onPressed: () {
                    showDateRangePicker(
                      context: context,
                      currentDate: DateTime.now(),
                      initialDateRange: DateTimeRange(
                        start: DateTime.now(),
                        end: DateTime.now().add(const Duration(days: 7)),
                      ),
                      firstDate: DateTime(2000),
                      lastDate: DateTime(2025),
                    );
                  },
                  child: const Text('Show Date Range Picker'),
                );
              }),
            ),
          ),
        );
      }
    }
    ```
    
    </details>
    
    ### When using `rangePickerHeaderBackgroundColor` in `DatePickerTheme` for M2
    
    ```dart
          theme: ThemeData(
            useMaterial3: false,
            datePickerTheme: const DatePickerThemeData(
              rangePickerHeaderBackgroundColor: Colors.red,
            ),
          ),
    ```
    
    | Before | After |
    | --------------- | --------------- |
    | <img src="https://github.com/flutter/flutter/assets/48603081/4e85a115-c702-45b2-8f1e-2f3c87c37629" /> | <img src="https://github.com/flutter/flutter/assets/48603081/d002ba0a-6328-4f97-8ba0-dcbb9662bbb3"  /> |
    TahaTesser authored Apr 26, 2024
    Configuration menu
    Copy the full SHA
    9a2d94e View commit details
    Browse the repository at this point in the history
  6. Bump peter-evans/create-pull-request from 6.0.4 to 6.0.5 (#147388)

    Bumps [peter-evans/create-pull-request](https://github.com/peter-evans/create-pull-request) from 6.0.4 to 6.0.5.
    <details>
    <summary>Release notes</summary>
    <p><em>Sourced from <a href="https://github.com/peter-evans/create-pull-request/releases">peter-evans/create-pull-request's releases</a>.</em></p>
    <blockquote>
    <h2>Create Pull Request v6.0.5</h2>
    <p>�� Fixes an issue with proxy support for users that run self-hosted behind a proxy.</p>
    <h2>What's Changed</h2>
    <ul>
    <li>fix: update proxy support to follow octokit change to fetch api by <a href="https://github.com/peter-evans"><code>@�peter-evans</code></a> in <a href="https://redirect.github.com/peter-evans/create-pull-request/pull/2867">peter-evans/create-pull-request#2867</a></li>
    </ul>
    <p><strong>Full Changelog</strong>: <a href="https://github.com/peter-evans/create-pull-request/compare/v6.0.4...v6.0.5">https://github.com/peter-evans/create-pull-request/compare/v6.0.4...v6.0.5</a></p>
    </blockquote>
    </details>
    <details>
    <summary>Commits</summary>
    <ul>
    <li><a href="https://github.com/peter-evans/create-pull-request/commit/6d6857d36972b65feb161a90e484f2984215f83e"><code>6d6857d</code></a> fix: update proxy support to follow octokit change to fetch api (<a href="https://redirect.github.com/peter-evans/create-pull-request/issues/2867">#2867</a>)</li>
    <li>See full diff in <a href="https://github.com/peter-evans/create-pull-request/compare/9153d834b60caba6d51c9b9510b087acf9f33f83...6d6857d36972b65feb161a90e484f2984215f83e">compare view</a></li>
    </ul>
    </details>
    <br />
    
    [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=peter-evans/create-pull-request&package-manager=github_actions&previous-version=6.0.4&new-version=6.0.5)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
    
    Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
    
    ---
    
    <details>
    <summary>Dependabot commands and options</summary>
    <br />
    
    You can trigger Dependabot actions by commenting on this PR:
    - `@dependabot rebase` will rebase this PR
    - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
    - `@dependabot merge` will merge this PR after your CI passes on it
    - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
    - `@dependabot cancel merge` will cancel a previously requested merge and block automerging
    - `@dependabot reopen` will reopen this PR if it is closed
    - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
    - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency
    - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
    - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
    - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    
    </details>
    dependabot[bot] authored Apr 26, 2024
    Configuration menu
    Copy the full SHA
    af34484 View commit details
    Browse the repository at this point in the history
  7. Fix FloatingActionButton docs for background and foreground pro…

    …perties (#147372)
    
    fixes [Material 3 `FloatingActionButton` background/foreground docs are inconsistent](#147368)
    TahaTesser authored Apr 26, 2024
    Configuration menu
    Copy the full SHA
    4f5ba5e View commit details
    Browse the repository at this point in the history
  8. Roll Flutter Engine from e99fc6ef91ef to 11a857e1599f (4 revisions) (#…

    …147399)
    
    flutter/engine@e99fc6e...11a857e
    
    2024-04-26 [email protected] Roll Dart SDK from 29afb0500d51 to e1bc6a6bca79 (1 revision) (flutter/engine#52405)
    2024-04-26 [email protected] Roll Fuchsia Linux SDK from PJBX8xxRnd5vCFnQM... to Rc4K-_-ym7vwUzXyo... (flutter/engine#52404)
    2024-04-25 [email protected] Remove "gclient sync" warning call during pre-rebase (flutter/engine#52342)
    2024-04-25 [email protected] Roll Dart SDK from f9c9894a2500 to 29afb0500d51 (1 revision) (flutter/engine#52399)
    
    Also rolling transitive DEPS:
      fuchsia/sdk/core/linux-amd64 from PJBX8xxRnd5v to Rc4K-_-ym7vw
    
    If this roll has caused a breakage, revert this CL and stop the roller
    using the controls here:
    https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
    Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
    is aware of the problem.
    
    To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
    
    To report a problem with the AutoRoller itself, please file a bug:
    https://issues.skia.org/issues/new?component=1389291&template=1850622
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
    engine-flutter-autoroll authored Apr 26, 2024
    Configuration menu
    Copy the full SHA
    241078a View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    5f0628d View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    2e80670 View commit details
    Browse the repository at this point in the history
Loading