feat(tabs): exposes new values in brn-tabs-trigger

thatsamsonkid commented 8 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
[x] 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
[ ] input
[ ] label
[ ] menubar
[ ] navigation-menu
[ ] popover
[ ] progress
[ ] radio-group
[ ] scroll-area
[ ] select
[ ] separator
[ ] sheet
[ ] skeleton
[ ] slider
[ ] switch
[ ] table
[x] tabs
[ ] textarea
[ ] toast
[ ] toggle
[ ] tooltip
[ ] typography

What is the current behavior?

Brn Trigger currently does not expose the underlying elementRef as well as some other key values that could be useful in customizing the tabs component.

See this as an example of code used to create below tabs example

https://github.com/thatsamsonkid/portfolio-analog/blob/main/src/app/content/experience.component.ts#L199

Closes #

What is the new behavior?

PR update Brn trigger directive to expose underlying elementRef as well getters for key and if the tab is currently active.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

https://github.com/goetzrobin/spartan/assets/22568206/c8bdaa91-9af5-49fe-acea-c98510c329ef

goetzrobin commented 8 months ago

Also, @thatsamsonkid can you add the exposedAs for this issue: https://github.com/goetzrobin/spartan/issues/105? Thanks already!

goetzrobin commented 8 months ago

Also, @thatsamsonkid can you add the exposedAs for this issue: https://github.com/goetzrobin/spartan/issues/105? Thanks already!

thatsamsonkid commented 8 months ago

All sounds like good ideas to me, I'll make the changes later today

goetzrobin / spartan