Skip to content

Some doc comments refer to unknown, or out of scope, code identifiers. #150562

Closed

Description

@srawlins

Here's the main list. Some just need to use backticks instead of square brackets. Some should just not have square brackets. Some are typos. Some need broader textual refactoring:

  • * DragGestureRecognizer.onCancel - property referencing allowedButtonsFilter.
  • * DragGestureRecognizer.onDown - property referencing allowedButtonsFilter.
  • * DragGestureRecognizer.onEnd - property referencing allowedButtonsFilter.
  • * DragGestureRecognizer.onStart - property referencing allowedButtonsFilter.
  • * DragGestureRecognizer.onUpdate - property referencing allowedButtonsFilter.
  • * DoubleTapGestureRecognizer.onDoubleTap - property referencing allowedButtonsFilter.
  • * DoubleTapGestureRecognizer.onDoubleTapCancel - property referencing referencing allowedButtonsFilter.
  • * DoubleTapGestureRecognizer.onDoubleTapDown - property referencing allowedButtonsFilter.
  • * TapGestureRecognizer - class referencing allowedButtonsFilter.
  • * Badge.backgroundColor - property referencing ColorScheme.errorColor.
  • * ButtonThemeData.padding - property referencing button.
  • * ButtonThemeData.shape - property referencing button.
  • * InkSparkle - class referencing containedInkWell.
  • * InkSparkle - class referencing rectCallback.
  • * InkSparkle.splashFactory - static const field referencing turbulenceSeed.
  • * InkSparkle.constantTurbulenceSeedSplashFactory - static const field referencing turbulenceSeed.
  • * Switch.adaptive - constructor referencing ThemeData.adaptations.
  • * TextButton.defaultStyleOf - method referencing padding.
  • * Adaptation.adapt - method referencing ThemeData.adaptations.
  • * UndoManager - Class referencing canRedo.
  • * UndoManager - Class referencing canUndo.
  • * ContextMenuController.markNeedsBuild - method referencing contextMenuBuilder.
  • * SliverFadeTransition - class referencing sliver.

And some just refer to not-properly-prefixed elements. These can be updated with a prefix:

  • * ImageInfo.clone - method referencing Image.clone.
  • * ImageInfo.isCloneOf - method referencing Image.isCloneOf.
  • * RawImage - class referencing Image.dispose.
  • * RawImage.image - property referencing Image.dispose.

And some just refer to not-imported elements. These can be updated with @docImport:

  • * DiagnosticsNode.toTimelineArguments - method referencing Timeline.startSync.
  • * compute - top-level function referencing Isolate.run.

Activity

  1. srawlins commented on Jun 20, 2024

    @srawlins
    ContributorAuthor
  2. added
    in triagePresently being triaged by the triage team
    frameworkflutter/packages/flutter repository. See also f: labels.
    d: api docsIssues with https://api.flutter.dev/
    and removed
    in triagePresently being triaged by the triage team
    on Jun 21, 2024
  3. goderbauer commented on Jun 24, 2024

    @goderbauer
    Member

    I am so excited! Turned the top-level comment into a checklist so we can keep track of what has been fixed. Do you want help with any of these?

  4. goderbauer commented on Jun 24, 2024

    @goderbauer
    Member

    Oh, looks like you already fixed them all in #150540. Thank you! Can we close this issue then or is there more to do?

  5. srawlins commented on Jun 24, 2024

    @srawlins
    ContributorAuthor

    Haha you read my mind. I was also going to turn it into a checklist. I think there is more work to be done:

    • DragGestureRecognizer.onCancel - property referencing allowedButtonsFilter.
    • DragGestureRecognizer.onDown - property referencing allowedButtonsFilter.
    • DragGestureRecognizer.onEnd - property referencing allowedButtonsFilter.
    • DragGestureRecognizer.onStart - property referencing allowedButtonsFilter.
    • DragGestureRecognizer.onUpdate - property referencing allowedButtonsFilter.
    • DoubleTapGestureRecognizer.onDoubleTap - property referencing allowedButtonsFilter.
    • DoubleTapGestureRecognizer.onDoubleTapCancel - property referencing referencing allowedButtonsFilter.
    • DoubleTapGestureRecognizer.onDoubleTapDown - property referencing allowedButtonsFilter.
    • TapGestureRecognizer - class referencing allowedButtonsFilter.
    • InkSparkle - class referencing containedInkWell.
    • InkSparkle - class referencing rectCallback.
    • InkSparkle.splashFactory - static const field referencing turbulenceSeed.
    • InkSparkle.constantTurbulenceSeedSplashFactory - static const field referencing turbulenceSeed.
    • Switch.adaptive - constructor referencing ThemeData.adaptations.
    • TextButton.defaultStyleOf - method referencing padding.
    • Adaptation.adapt - method referencing ThemeData.adaptations.
    • UndoManager - Class referencing canRedo.
    • UndoManager - Class referencing canUndo.
    • ContextMenuController.markNeedsBuild - method referencing contextMenuBuilder.

    For all of these, I just changed [foo] into `foo`, so they're no-op improvements as far as the rendered HTML. I think many of them refer to parameters on other methods, like a class comment refering to a parameter on a constructor. I couldn't really say, which is why I landed the no-op fixes.

  6. goderbauer commented on Jun 24, 2024

    @goderbauer
    Member

    Ah, Ok, I am fixing those (where it made sense) in #150734. Some actually need to be in `. I'll mark that PR as fixing this bug then.

  7. github-actions commented on Jul 9, 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.

  8. locked as resolved and limited conversation to collaborators on Jul 9, 2024
  9. added a commit that references this issue on Apr 20, 2026
  10. added a commit that references this issue on Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    d: api docsIssues with https://api.flutter.dev/frameworkflutter/packages/flutter repository. See also f: labels.team-frameworkOwned by Framework team

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions