goetzrobin / spartan

Cutting-edge tools powering Angular full-stack development.
https://spartan.ng
MIT License
1.29k stars 137 forks source link

fix(accordion): remove tailwind from brn #97

Closed elite-benni closed 8 months ago

elite-benni commented 8 months ago

The brain should not use tailwind. Brain should also be usable in an project without tailwind.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

Which package are you modifying?

What is the current behavior?

The accordion-content does hide on a project where tailwind is not used, or where no overflow-hidden is used.

Closes #

What is the new behavior?

The accordion-content does not rely on tailwind anymore, the overflow is set by as direct style instead of tailwind overflow-hidden.

Does this PR introduce a breaking change?

Other information

goetzrobin commented 8 months ago

@elite-benni let's fix this as part of this PR: https://github.com/goetzrobin/spartan/pull/89