Skip to content

Commit

Permalink
feat(list)!: require label for context when dragging between lists (#…
Browse files Browse the repository at this point in the history
…10702)

**Related Issue:** #7537

## Summary
- Modifies the component's `label` property to be required
- Provides context to the `label` property, where its value specifies
the name when dragging between lists discussed in
#7537 (comment)
and the subsequent thread

BREAKING CHANGE: The component's `label` property is required to provide
context between lists when dragging.
  • Loading branch information
geospatialem authored Nov 15, 2024
1 parent 5cf4cb5 commit 5b84856
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions packages/calcite-components/src/components/list/list.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -219,8 +219,12 @@ export class List
*/
@property({ reflect: true }) interactionMode: InteractionMode = "interactive";

/** Specifies an accessible name for the component. */
@property() label: string;
/**
* Specifies an accessible name for the component.
*
* When `dragEnabled` is `true` and multiple list sorting is enabled with `group`, specifies the component's name for dragging between lists.
*/
@property() label!: string;

/** When `true`, a busy indicator is displayed. */
@property({ reflect: true }) loading = false;
Expand Down

0 comments on commit 5b84856

Please sign in to comment.