Skip to content

Suggestion: rename DestinationLabel to BottomNavigationBarItem (or similar) #7278

Closed

Description

@lukef

DestinationLabel seems (?) to be a term used in the Material Design docs but when initially onboarding it can be confusing as a concept because they contain more than a simple label. In addition, the property for setting the items in the navbar is labels so it should probably be renamed to items.

DestinationLabel seems to be only used in the BottomNavigationBar but I'm not sure if there is a broader use case I'm missing.

Happy to do a PR if you need.

Activity

  1. added
    frameworkflutter/packages/flutter repository. See also f: labels.
    p: material_uimaterial_ui package in flutter/packages
    c: API breakBackwards-incompatible API changes
    on Dec 16, 2016
  2. abarth commented on Dec 16, 2016

    @abarth
    Contributor

    /cc @Hixie

  3. abarth commented on Dec 16, 2016

    @abarth
    Contributor
  4. Hixie commented on Dec 16, 2016

    @Hixie
    Contributor

    There was another thing we were going to use them for... Can't remember what though. :-)

  5. HansMuller commented on Dec 16, 2016

    @HansMuller
    Contributor

    DestinationLabel and TabLabel are rather similar, however in the new version of tabs I'm working on, TabLabel isn't needed anymore. I agree that BottomNavigationBarItem is a more obvious name for the items in a BottomNavigationBar, despite the material design spec.

  6. lukef commented on Dec 16, 2016

    @lukef
    ContributorAuthor

    I created a PR (#7281) for this if you want to review.

  7. abarth commented on Jan 4, 2017

    @abarth
    Contributor

    Looks like this happened.

  8. github-actions commented on Sep 5, 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.

  9. locked as resolved and limited conversation to collaborators on Sep 5, 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

    c: API breakBackwards-incompatible API changesframeworkflutter/packages/flutter repository. See also f: labels.p: material_uimaterial_ui package in flutter/packages

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions