Skip to content

[url_launcher] Replace deprecated renderView #143449

Closed
flutter/packages
#6137

Description

@stuartmorgan-g
'renderView' is deprecated and shouldn't be used. Consider using
RendererBinding.renderViews instead as the binding may manage multiple RenderViews. This feature was
deprecated after v3.10.0-12.0.pre. Try replacing the use of the deprecated member with the replacement.

I'm not sure if this is in 3.10 or 3.13 from the message; currently it's test-only code, so we could wait until 3.13 is our min-min version to fix this, since test-only fixes don't need to spread through the client ecosystem.

Activity

  1. added
    p: url_launcherPlugin to launch external applications
    packageflutter/packages repository. See also p: labels.
    c: tech-debtTechnical debt, code quality, testing, etc.
    p: deprecated apiUse of a deprecated Dart API in flutter/packages that needs to be updated
    on Feb 14, 2024
  2. stuartmorgan-g commented on Feb 15, 2024

    @stuartmorgan-g
    ContributorAuthor

    Looks like url_launcher is already 3.13-min.

  3. stuartmorgan-g commented on Feb 15, 2024

    @stuartmorgan-g
    ContributorAuthor

    We actually have a non-test usage of this as well, but it's not being flagged since it's in the deprecated url_launcher API. I'm not sure there's much we can do other than delete that API to fix the use of the deprecated API, because launch doesn't have enough information to know which renderer to set the system UI overlay style for.

    The API was deprecated 22 months ago in 6.1.0, so I think removing it at this point is reasonable. There are >800 published packages depending on url_launcher though, so there's still the potential for some ecosystem effects.

  4. stuartmorgan-g commented on Feb 15, 2024

    @stuartmorgan-g
    ContributorAuthor

    After some more investigation through the API surface, there's a way to get best-effort backwards compatible behavior that will fall back to a no-op in the worst case, using non-deprecated APIs, so I'll do that to avoid a breaking change at this point.

  5. stuartmorgan-g commented on Feb 21, 2024

    @stuartmorgan-g
    ContributorAuthor

    Looks like it's actually 3.16, but since it's not test-only after all I'm going to drop 3.13 for this.

  6. github-actions commented on Mar 6, 2024

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  7. locked as resolved and limited conversation to collaborators on Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

c: tech-debtTechnical debt, code quality, testing, etc.p: deprecated apiUse of a deprecated Dart API in flutter/packages that needs to be updatedp: url_launcherPlugin to launch external applicationspackageflutter/packages repository. See also p: labels.team-ecosystemOwned by Ecosystem team

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions