Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(list): enable dragging on list items contained within a list that supports dragEnabled #9660

Merged
merged 7 commits into from
Jun 21, 2024

Conversation

driskull
Copy link
Member

@driskull driskull commented Jun 21, 2024

Related Issue: #9662

Summary

I'd like to get this one in to support work in Maps SDK.

A limitation of the Maquette VDOM is that it creates a "div" for every new widget. So in order to make a list item as a widget it needs to support calcite-list-items wrapped in divs.

  • Support dragging on list-items that are not direct children of a list
  • Add tests

@github-actions github-actions bot added the bug Bug reports for broken functionality. Issues should include a reproduction of the bug. label Jun 21, 2024
@driskull driskull marked this pull request as ready for review June 21, 2024 18:43
@driskull driskull requested a review from a team as a code owner June 21, 2024 18:43
Copy link
Member

@jcfranco jcfranco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏
🤏↕️🤏🤏🤏↕️↕️↕️↕️🤏↕️↕️↕️↕️↕️🤏↕️🤏🤏🤏↕️🤏↕️🤏
🤏↕️🤏🤏🤏↕️🤏🤏🤏🤏🤏🤏↕️🤏🤏🤏↕️↕️🤏↕️↕️🤏↕️🤏
🤏↕️🤏🤏🤏↕️🤏↕️↕️🤏🤏🤏↕️🤏🤏🤏↕️🤏↕️🤏↕️🤏↕️🤏
🤏↕️🤏🤏🤏↕️🤏🤏↕️🤏🤏🤏↕️🤏🤏🤏↕️🤏🤏🤏↕️🤏🤏🤏
🤏↕️↕️↕️🤏↕️↕️↕️↕️🤏🤏🤏↕️🤏🤏🤏↕️🤏🤏🤏↕️🤏↕️🤏
🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏🤏

…/calcite-design-system into dris0000/list-fix-scope-dragEnabled
@driskull driskull added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label Jun 21, 2024
@driskull driskull merged commit 5010ef9 into dev Jun 21, 2024
12 checks passed
@driskull driskull deleted the dris0000/list-fix-scope-dragEnabled branch June 21, 2024 19:27
@github-actions github-actions bot added this to the 2024-06-25 - Jun Release milestone Jun 21, 2024
benelan added a commit that referenced this pull request Jun 24, 2024
…rovements

* origin/dev:
  build(deps): update typescript-eslint monorepo to v7.13.1 (#9657)
  docs: update component READMEs (#9668)
  chore: release next
  chore: minor git tooling tweaks (#9664)
  feat(combobox): add `filterText` prop (#9654)
  chore: release next
  feat(list): add filterProps property to specify which properties to filter against (#9622)
  chore: release next
  fix(list): enable dragging on list items contained within a list that supports dragEnabled (#9660)
  test(filter): add coverage for empty filter text (#9655)
  chore: release next
  fix: set the correct Sortable parent when within a shadowRoot (#9532)
  fix(combobox): allow arrow selection of entered text (#9629)
@benelan benelan added autorelease: tagged Pull requests created by release-please that were merged and deployed and removed autorelease: tagged Pull requests created by release-please that were merged and deployed labels Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug reports for broken functionality. Issues should include a reproduction of the bug. pr ready for visual snapshots Adding this label will run visual snapshot testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants