Skip to content

Update chip.dart to use set of MaterialState #128289

@TahaTesser

Description

@TahaTesser

Is there an existing issue for this?

Use case

Currently, two classes in the framework use MaterialStateMixin

  1. material/chip.dart
  2. material/button.dart (to be deprecated widget)

While working on #115827, I noticed this.
Every other class in the framework uses a set of MaterialState, which is updated for different states within the widget.

Proposal

My proposal is, either to make chip.dart consistent with the rest of the framework to maintain material states by removing MaterialStateMixin and using a set of material state
or
continue using MaterialStateMixin and consider why MaterialStateMixin is needed.

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2Important issues not at the top of the work listc: tech-debtTechnical debt, code quality, testing, etc.f: material designflutter/packages/flutter/material repository.frameworkflutter/packages/flutter repository. See also f: labels.r: fixedIssue is closed as already fixed in a newer versionteam-designOwned by Design Languages teamtriaged-designTriaged by Design Languages team

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions