Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

How looks like a typed BEM helper method approach #7134

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

deleonio
Copy link
Contributor

@deleonio deleonio commented Nov 28, 2024

The A11y and PO reviews will only take place after all other DoD steps have been completed by the Developer:

  • Meaningful pull request title for the release notes
  • Pull request is linked to an issue and all changes relate to the issue
  • Tests to protect this code implemented (if applicable)
  • Manual test performed successfully (if applicable)
  • Documentation or migration has been updated (if applicable)

Copy link
Contributor

github-actions bot commented Nov 28, 2024

@deleonio deleonio force-pushed the dry/typed-bem branch 2 times, most recently from d00b561 to 0ea3dd5 Compare November 28, 2024 06:48
@deleonio deleonio marked this pull request as draft November 28, 2024 07:31
@deleonio deleonio changed the title How looks like a typed bem helper method approach How looks like a typed BEM helper method approach Nov 28, 2024
@deleonio deleonio force-pushed the dry/typed-bem branch 4 times, most recently from a1e4b99 to 1f7fab9 Compare November 30, 2024 09:52
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das könnte für Dich @Makko74 interessant sein.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich fand es beim ersten lesen recht kompliziert, bis ich in der Doku gesehen habe, dass du scss files generieren möchtest.

Im ersten Schritt sehe in keinen Mehrwert Css Klassennamen typ sicher zu machen , da alles an einer Stelle stattfindet.

Wenn man das Generieren von scss Dateien mit betrachtet, macht es schon sinn. Aber auf dem ersten Blick fällt mir auf, dass es nicht Refactoring tauglich ist. Ändere ich in der Alert Datei irgendwas ab kann ich die scss Datei nicht so ohne weiteres mehr automatisch updaten, da mir sonst die css Anweisungen verloren gehen.
Dies bedeutet ich benötige eher einen js-to-scss-with-bem Ansatz (abgeleitet con js-to-css. ;) ) Dann kann die Datei automatisch immer gelöscht und generiert werden. Und es ist Refactoring tauglich.

scss mixin macht die Sache aber zusätzlich kompliziert.

Wenn diese natürlich so funktionieren könnte, wäre es dann recht angenehm die Themes dazu zu generieren. Da man den Typ dann importieren kann und via js die css Anweisungen schreiben kann.

Ich würde aber davon abraten 'easy-bem' oder ein anderes Paket zu nutzen, der Code ist recht klein keine 20 Zeilen und dafür lohn es sich nicht in Abhängigkeit zu einem anderen Paket zu gehen.

Im Component Projekt würde ich persönlich trotzdem eine Art Wrapper hinzufügen, der dann typed-bem und csx kapselt.

ich bleibe aber trotzdem kein Fan von BEM, da im SSR Modus der Output sich extrem aufblähen wird.

@deleonio deleonio added the v3 Here are issues that need to be resolved for version 3. label Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v3 Here are issues that need to be resolved for version 3.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants