Skip to content

Commit

Permalink
fix: expose data-state on item-text for combobox and select (#1621)
Browse files Browse the repository at this point in the history
  • Loading branch information
Jonas-C authored Jun 26, 2024
1 parent 29051fd commit 1f7ea87
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 0 deletions.
7 changes: 7 additions & 0 deletions .changeset/seven-buttons-drum.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
"@zag-js/combobox": patch
"@zag-js/select": patch
"@zag-js/docs": patch
---

Expose data-state on ItemText for Combobox and Select
2 changes: 2 additions & 0 deletions packages/docs/data/data-attr.json
Original file line number Diff line number Diff line change
Expand Up @@ -319,6 +319,7 @@
"ItemText": {
"data-scope": "combobox",
"data-part": "item-text",
"data-state": "\"checked\" | \"unchecked\"",
"data-disabled": "Present when disabled",
"data-highlighted": "Present when highlighted"
},
Expand Down Expand Up @@ -907,6 +908,7 @@
"ItemText": {
"data-scope": "select",
"data-part": "item-text",
"data-state": "\"checked\" | \"unchecked\"",
"data-disabled": "Present when disabled",
"data-highlighted": "Present when highlighted"
},
Expand Down
1 change: 1 addition & 0 deletions packages/machines/combobox/src/combobox.connect.ts
Original file line number Diff line number Diff line change
Expand Up @@ -390,6 +390,7 @@ export function connect<T extends PropTypes, V extends CollectionItem>(
return normalize.element({
...parts.itemText.attrs,
dir: state.context.dir,
"data-state": itemState.selected ? "checked" : "unchecked",
"data-disabled": dataAttr(itemState.disabled),
"data-highlighted": dataAttr(itemState.highlighted),
})
Expand Down
1 change: 1 addition & 0 deletions packages/machines/select/src/select.connect.ts
Original file line number Diff line number Diff line change
Expand Up @@ -281,6 +281,7 @@ export function connect<T extends PropTypes, V extends CollectionItem = Collecti
const itemState = getItemState(props)
return normalize.element({
...parts.itemText.attrs,
"data-state": itemState.selected ? "checked" : "unchecked",
"data-disabled": dataAttr(itemState.disabled),
"data-highlighted": dataAttr(itemState.highlighted),
})
Expand Down

0 comments on commit 1f7ea87

Please sign in to comment.