Skip to content
This repository was archived by the owner on Aug 12, 2024. It is now read-only.
This repository was archived by the owner on Aug 12, 2024. It is now read-only.

Standard-/Vaadin-conform public API #12

@kevinkendzia

Description

@kevinkendzia

The fluent-"with"-methods of the ChipCombobox class are nice to build a fully configured combobox in one line of code, but I think we are missing the default setters.
Especially if you're setting single values dynamically, a setter in the form "setXYZ" more clear than "withXYZ". Also the first method I search for to set a value is a default setter.

// Current
cbo.withLabel("TEST");

// request
cbo.setLabel("TEST");

Please ADD default setters (don't remove the "with"-Setter)

Also for the withAllAvailableItems()-methods consider renaming (at least the default setter) to just "setItems()" and implement the Vaadin-Interface HasItems to be Vaadin-conform.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions