feat(command): allow items to be dynamically selected

[stewones]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our
  guidelines: https://github.com/goetzrobin/spartan/blob/main/CONTRIBUTING.md#-commit-message-guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• Other... Please describe:

Which package are you modifying?

• accordion
• alert
• alert-dialog
• aspect-ratio
• avatar
• badge
• button
• calendar
• card
• checkbox
• collapsible
• combobox
• command
• context-menu
• data-table
• date-picker
• dialog
• dropdown-menu
• hover-card
• icon
• input
• label
• menubar
• navigation-menu
• pagination
• popover
• progress
• radio-group
• scroll-area
• select
• separator
• sheet
• skeleton
• slider
• sonner
• spinner
• switch
• table
• tabs
• textarea
• toast
• toggle
• tooltip
• typography

What is the current behavior?

currently, there's no way to tell if a command item is aria-selected. so the problem is that the first item will always look like this:

What is the new behavior?

after this PR it would look like this:

  <brn-cmd-list hlm>
    <brn-cmd-group hlm>
       @for (item of items(); track $index) {
         <button
              hlm
              brnCmdItem
              [selected]="currentItem() === item"
            >

Does this PR introduce a breaking change?

• Yes
• No

Other information

[stewones]

is the ci/prettier something I can fix? cause locally ci.format-sort says there's no fix to apply.

[stewones]

I realized there must be a bug with the cmdk lib, theoretically to achieve what I want here a simple class cmdk-item-active should be enough but I have no idea why it doesn't work.

[stewones]

ended up with a simpler solution, I tried to avoid this change but looks like setting aria-selected on the host only has no effect.

The final result looks like this, I wish we could get rid of the first item always being aria-selected no matter what, but can't see a way out of this without breaking the keyboard navigation (cmdk)

Screen.Recording.2024-10-16.at.20.46.03.mov