-
Notifications
You must be signed in to change notification settings - Fork 35
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
base: develop
Are you sure you want to change the base?
Conversation
Netlify Draft Deployment |
d00b561
to
0ea3dd5
Compare
a1e4b99
to
1f7fab9
Compare
1f7fab9
to
950de34
Compare
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
The A11y and PO reviews will only take place after all other DoD steps have been completed by the Developer: