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: 42fb0b2
Choose a base ref
...
head repository: flutter/flutter
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 3476b96
Choose a head ref
  • 20 commits
  • 55 files changed
  • 13 contributors

Commits on Apr 18, 2023

  1. Configuration menu
    Copy the full SHA
    55502fc View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    9b53029 View commit details
    Browse the repository at this point in the history
  3. l10n.yaml's nullable-getter option should default to true (#124353)

    Currently, nullable-getter defaults to false when l10n.yaml is not present, which is not the same behavior as when an l10n.yaml file is present and nullable-getter is not set.
    
    Fixes #120457.
    thkim1011 authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    b6c7df4 View commit details
    Browse the repository at this point in the history
  4. Remove unused getRootRenderObject and getSelectedRenderObject service…

    … extensions (#124805)
    
    These are currently unused and will not work in a multi-view environment where we will have multiple render trees.
    
    Work towards #121573.
    goderbauer authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    df811c6 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    ad784bf View commit details
    Browse the repository at this point in the history
  6. [web] - Clean up skipped tests (#124981)

    Fixes #124174
    Fixes #124980
    htoor3 authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    cb90ae4 View commit details
    Browse the repository at this point in the history
  7. Remove impeller testowners (#125056)

    Remove TESTOWNERS matching the removed impeller tests in #125044.
    
    Filed #125055 since I believe the mismatch should have been caught in that PR.
    jmagman authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    0ea2f3b View commit details
    Browse the repository at this point in the history
  8. Limit the number of Material spell check suggestions to 3 (#124899)

    Fixes a bug where the spell check menu could overflow.
    justinmc authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    f04a5af View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    98dfdf1 View commit details
    Browse the repository at this point in the history
  10. Add controller argument to SubmenuButton (#125000)

    ## Description
    
    This adds an optional argument to the `SubmenuButton` that allows the creator to supply a `MenuController` for controlling the menu.
    
    ## Related Issues
     - Fixes #124988
    
    ## Tests
     - Added tests for new argument.
    gspencergoog authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    4b39f07 View commit details
    Browse the repository at this point in the history
  11. Update to add Kim Jiun to AUTHORS (#125026)

    Adding my name (Kim Jiun) to Authors.
    
    REF : #91496
    Thanks!
    KKimj authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    e2a2046 View commit details
    Browse the repository at this point in the history
  12. Remove double.fromEnvironment from dart-define doc (#124102)

    Remove double.fromEnvironment from dart-define doc.
    
    Fixes #124665 
    
    The current dart-define docs reads:
    
    ```
    Additional key-value pairs that will be available as constants from the String.fromEnvironment, bool.fromEnvironment, int.fromEnvironment, and
    double.fromEnvironment constructors.
    ```
    while dart-define-from-file reads:
    
    ```
    The path of a json format file where flutter define a global constant pool. 
    Json entry will be available as constants from the String.fromEnvironment, bool.fromEnvironment, int.fromEnvironment, 
    and double.fromEnvironment constructors; the key and field are json values
    ```
    
    <img width="1440" alt="Screenshot 2023-04-03 at 19 31 34" src="https://user-images.githubusercontent.com/31275429/229720102-2020e1a9-c1fb-470b-89e6-ffb84096a51b.png">
    
    However, there is no fromEnvironment constructor for double. The aim of this PR is to address this by removing double.fromEnvironment from help docs as stated in
    Mastersam07 authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    81c3bc5 View commit details
    Browse the repository at this point in the history
  13. Remove package:js/dart:js_interop conflicts (#124879)

    dart:js_interop and package:js will start conflicting, since they both have an `@JS` annotation. Until we're ready to only use dart:js_interop (which will require updating the SDK constraints of every package), we should hide the `@JS` annotation from dart:js_interop. Due to shadowing, this is the behavior today, so there should be no functional change.
    
    Unblocks https://dart-review.googlesource.com/c/sdk/+/294130/8 and prevents confusing shadowing of dart:js_interop annotations like we do today.
    
    - [Mentioned CL that is unblocked] I listed at least one issue that this PR fixes in the description above.
    - [Need test-exemption] I added new tests to check the change I am making, or this PR is [test-exempt].
    - [Need to run] All existing and new tests are passing.
    srujzs authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    6edbc19 View commit details
    Browse the repository at this point in the history
  14. Roll Flutter Engine from 6d263ea56a62 to 5fcc7b719029 (4 revisions) (#…

    …125069)
    
    flutter/engine@6d263ea...5fcc7b7
    
    2023-04-18 [email protected] [Impeller] Gpu model information to Skia gold (flutter/engine#41216)
    2023-04-18 [email protected] [Impeller] faster glyph atlas generation by removing data copies (flutter/engine#41290)
    2023-04-18 [email protected] Migrate android AOT to engine_v2. (flutter/engine#41229)
    2023-04-18 [email protected] Roll Skia from 5bd4bdc0d8e2 to f80ee1088861 (8 revisions) (flutter/engine#41302)
    
    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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
    engine-flutter-autoroll authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    c05bc40 View commit details
    Browse the repository at this point in the history
  15. Adaptive alert dialog (#124336)

    Fixes #102811. Adds an adaptive constructor to AlertDialog, along with the adaptive function showAdaptiveDialog.
    
    <img width="357" alt="Screenshot 2023-04-06 at 10 40 18 AM" src="https://user-images.githubusercontent.com/58190796/230455412-31100922-cfc5-4252-b8c6-6f076353f29e.png">
    <img width="350" alt="Screenshot 2023-04-06 at 10 42 50 AM" src="https://user-images.githubusercontent.com/58190796/230455454-363dd37e-c44e-4aca-b6a0-cfa1d959f606.png">
    MitchellGoodwin authored Apr 18, 2023
    Configuration menu
    Copy the full SHA
    bd2617e View commit details
    Browse the repository at this point in the history

Commits on Apr 19, 2023

  1. Disableable ContextMenuButtonItems (#124253)

    Fixes #124247
    
    | Native | Flutter before | Flutter after |
    | --- | --- | --- |
    | <img width="248" alt="Screenshot 2023-04-05 at 9 26 16 AM" src="https://user-images.githubusercontent.com/389558/230177116-154999e8-eef3-441d-9fe9-7063839a6b99.png"> | <img width="240" alt="Screenshot 2023-04-05 at 11 18 01 AM" src="https://user-images.githubusercontent.com/389558/230177125-1680e851-223e-4956-b5b6-1a24e11dc22a.png"> | <img width="226" alt="Screenshot 2023-04-05 at 11 17 36 AM" src="https://user-images.githubusercontent.com/389558/230177123-bde82134-67e1-4ce2-8eec-719eeb779bf4.png"> |
    
    Also, it's now possible for anyone to create disabled buttons like this by setting ContextMenuButtonItem.onPressed to `null`.
    justinmc authored Apr 19, 2023
    Configuration menu
    Copy the full SHA
    3128df8 View commit details
    Browse the repository at this point in the history
  2. Roll Flutter Engine from 5fcc7b719029 to 099ed6c62d04 (3 revisions) (#…

    …125078)
    
    flutter/engine@5fcc7b7...099ed6c
    
    2023-04-18 [email protected] Roll Fuchsia Mac SDK from qZHSvkpAU1-YYGvYc... to yD5a3QBJHUFM4nVou... (flutter/engine#41314)
    2023-04-18 [email protected] [web:canvaskit] clean up the rest of skia_object_cache usages (flutter/engine#41259)
    2023-04-18 [email protected] Roll Skia from f80ee1088861 to 476ee3fd54cc (2 revisions) (flutter/engine#41309)
    
    Also rolling transitive DEPS:
      fuchsia/sdk/core/mac-amd64 from qZHSvkpAU1-Y to yD5a3QBJHUFM
    
    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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
    engine-flutter-autoroll authored Apr 19, 2023
    Configuration menu
    Copy the full SHA
    d9392f2 View commit details
    Browse the repository at this point in the history
  3. Roll Flutter Engine from 099ed6c62d04 to 8b7cdb02f7f3 (6 revisions) (#…

    …125094)
    
    flutter/engine@099ed6c...8b7cdb0
    
    2023-04-19 [email protected] Roll dart, zlib, emscripten, tweak license script for rolled zlib (flutter/engine#41299)
    2023-04-19 [email protected] Roll Skia from 84630c3c18b2 to 5a718d9e9c06 (3 revisions) (flutter/engine#41318)
    2023-04-19 [email protected] [Impeller] Provide an opaque interface for color sources in Aiks (flutter/engine#41315)
    2023-04-18 [email protected] [web:canvaskit] remove unnecessary instrumentation from picture (flutter/engine#41313)
    2023-04-18 [email protected] Roll Skia from 476ee3fd54cc to 84630c3c18b2 (5 revisions) (flutter/engine#41316)
    2023-04-18 [email protected] Double zip FlutterMacOS.framework.zip. (flutter/engine#41306)
    
    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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
    engine-flutter-autoroll authored Apr 19, 2023
    Configuration menu
    Copy the full SHA
    ade9487 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    5923075 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    3476b96 View commit details
    Browse the repository at this point in the history
Loading