calcite-design-system icon indicating copy to clipboard operation
calcite-design-system copied to clipboard

fix(list-item): Always reserve spacing for open icon

Open driskull opened this issue 1 year ago • 7 comments

Related Issue: #8667

Summary

  • Always reserve space for open toggle icon
  • Screenshot tests show changes

driskull avatar Jan 30 '24 00:01 driskull

I just noticed that the "Nested List" page isn't expanded in the baseline or new so chromatic isn't picking up the changes. It might be worth expanding all the items in that page so we have it as a baseline.

ashetland avatar Jan 30 '24 19:01 ashetland

Also the baseline is showing the old icons? CleanShot 2024-01-30 at 11 02 14@2x

ashetland avatar Jan 30 '24 19:01 ashetland

I think those icons are slotted in, not in the component

driskull avatar Jan 30 '24 19:01 driskull

@ashetland is this one good to merge?

driskull avatar Feb 09 '24 01:02 driskull

Sadly no. We got feedback that folks aren't super happy about the added space in a flat list scenario. I'm drawing up specs for revised spacing in both flat and nested lists as there was a strong desire for this to be configurable. We'll need to push implementation to March. cc @jcfranco

ashetland avatar Feb 09 '24 16:02 ashetland

there was a strong desire for this to be configurable

I think that would be a breaking change if we required people to configure allowing nesting and doesn't seem very user friendly to have to do.

If the reserved space for a nesting icon is an issue maybe it should be moved to the opposite side like the block/accordion components?

driskull avatar Feb 09 '24 19:02 driskull

This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions.

github-actions[bot] avatar Feb 23 '24 01:02 github-actions[bot]