Borders inconsistent when list comprises both grouped and ungrouped list items

[nwhittaker]

Check existing issues

• [X] I have checked for existing issues to avoid duplicates

Actual Behavior

Given a list containing a mix of list items and grouped list items:

1. A border does not appear before an item if the previous item is in a group.
2. A border does appear before a group heading if the previous item is not in a group.

Expected Behavior

I expected borders to behave as they do in non-hybrid lists:

1. A border always appears before an item unless it's the first item.
2. A border never appears before a group heading.

Reproduction Sample

https://codepen.io/nwhittaker-esri/pen/LYMYpGj

Reproduction Steps

1. Visit reproduction sample
2. Verify no border appears between the Waterfalls and Rivers items
3. Verify a border appears before the Building group header

Reproduction Version

1.6.0

Relevant Info

No response

Regression?

No response

Priority impact

p4 - not time sensitive

Impact

No response

Calcite package

• [X] @esri/calcite-components
• [ ] @esri/calcite-components-react

Esri team

ArcGIS Field Apps

[github-actions[bot]]

Installed and assigned for verification.

[geospatialem]

Verified on the main branch:

[nwhittaker]

@geospatialem, I updated the repro sample to 1.11.0. I see the border is gone from above the Buildings header, but I don't see a border between the Waterfalls and Rivers list items. Fair enough if this is the expected design, but I wanted to double-check…

[anveshmekala]

@geospatialem, I updated the repro sample to 1.11.0. I see the border is gone from above the Buildings header, but I don't see a border between the Waterfalls and Rivers list items. Fair enough if this is the expected design, but I wanted to double-check…

Hi @nwhittaker , the border between list-item in a group and other list-item should be present (In this case Waterfalls and Rivers). This is a bug and will fix it. Thanks for reporting.

[github-actions[bot]]

Installed and assigned for verification.

[anveshmekala]

Hi @nwhittaker , the fix is installed in 1.11.1-next.2. Could you please verify if it is working as expected? Thank you.

[nwhittaker]

Hi @nwhittaker , the fix is installed in 1.11.1-next.2. Could you please verify if it is working as expected? Thank you.

@anveshmekala, I forked a new code sample and updated it to 1.11.1-next.2 and it looks good to me, thanks!

[geospatialem]

Closing per the verification above, also confirmed in 1.12.0-next.6. 🎉