feat(select,spinner,separator,toggle): more input signals

Celtian commented 3 months ago

PR Checklist

Please check if your PR fulfills the following requirements:

[x] 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)
[x] 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
[x] select
[x] separator
[ ] sheet
[ ] skeleton
[ ] slider
[ ] sonner
[x] spinner
[ ] switch
[ ] table
[ ] tabs
[ ] textarea
[ ] toast
[x] toggle
[ ] tooltip
[ ] typography

What is the current behavior?

Closes #

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

ajitzero commented 3 months ago

Hi, are you converting these manually? There is a reasonably stable CLI migration command for this.

Please take a look: https://twitter.com/Enea_Jahollari/status/1773335882780557471

npm i ngxtension
ng g ngxtension:convert-outputs

Celtian commented 3 months ago

Hi, are you converting these manually? There is a reasonably stable CLI migration command for this.

Please take a look: https://twitter.com/Enea_Jahollari/status/1773335882780557471
npm i ngxtension
ng g ngxtension:convert-outputs

But there is a problem. Compatibility with Storybook. You can's convert everything right now...

ajitzero commented 3 months ago

Ah, that makes sense! Thanks for the info, I hadn't tried it myself yet.

elite-benni commented 3 months ago

LGTM! Its just so much cleaner this way!

goetzrobin / spartan