Skip to content

Fill isn't extendableΒ #3140

Closed
Closed
@jjatie

Description

Currently Fill relies on an enum to determine which fill it is going to use. Since enums cannot have cases added to them, consumers cannot extend the FillType to add new types of chart fills. The current fillPath method would also need to be changed.

To fix this I propose creating a protocol all Fill types must adhere to as implemented in #3084. This allows users to create any Fill they desire in their apps, and don't require changes to be made to the framework. This also removes any confusion of unused variables as the current implementation requires all possible variables for every type in the same class.

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions