Skip to content

[doc] BottomNavigationBarItem : Update title with label to reflect correct message.  #67185

Closed

Description

@awhitford

I noticed that the API documentation for BottomNavigationBarItem is out of date.

It mentions:

The argument icon should not be null and the argument title should not be null when used in a Material Design's BottomNavigationBar.

However, since title is now deprecated, shouldn't this message be revised to say label instead?

Activity

  1. Abhishek01039 commented on Oct 3, 2020

    @Abhishek01039
    Contributor

    Hello @awhitford
    Thanks for opening an issue.
    I have created PR from your issue which you can check.
    if you have any suggestions then please let me know
    Thanks.

  2. darshankawar commented on Oct 5, 2020

    @darshankawar
    Member

    Although the implementation does say to use label instead, the concerned line could be updated.

    https://api.flutter.dev/flutter/widgets/BottomNavigationBarItem/BottomNavigationBarItem.html

  3. changed the title [-][doc] BottomNavigationBarItem[/-] [+][doc] BottomNavigationBarItem : Update title with label to reflect correct message. [/+] on Oct 5, 2020
  4. HansMuller commented on Oct 5, 2020

    @HansMuller
    Contributor
  5. Abhishek01039 commented on Oct 10, 2020

    @Abhishek01039
    Contributor

    Hi @awhitford
    now you can close the issue because PR has been merged.
    Now you can check here https://api.flutter.dev/flutter/widgets/BottomNavigationBarItem/BottomNavigationBarItem.html
    Thanks

  6. awhitford commented on Oct 27, 2020

    @awhitford
    Author

    @Abhishek01039 The BottomNavigationBarItem doc still references title (deprecated) instead of label.

    The argument icon should not be null and the argument title should not be null when used in a Material Design's BottomNavigationBar.

    This issue does not look resolved to me.

  7. github-actions commented on Aug 10, 2021

    @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 Aug 10, 2021
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.p: material_uimaterial_ui package in flutter/packages

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions