Vorschlag für ein anderes Modifier-Konzept #15
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
+ adds the is- and has- prefix improving recognation of modifiers + represents the current standard in zweitag projects + allows a wider range of modifiers (e.g. has-submenu instead of submenued)
jkimmeyer
requested review from
00cc00,
ChristianPeters,
SimonEggert,
jjuchim and
oliverliebsch
ChristianPeters
requested changes
Nov 5, 2020
| #### Eigenschaften des Namens | ||
|
|
||
| * beschreibt einen Zustand oder besonderes Verhalten einer Komponenteninstanz, wodurch eine Variante entsteht | ||
| * ist idealerweise ein einfaches Adjektiv |
There was a problem hiding this comment.
Das stimmt ja jetzt nur noch halb. Bitte definieren, wann is (Adjektiv) und wann has (Nomen)
| * beschreibt einen Zustand oder besonderes Verhalten einer Komponenteninstanz, wodurch eine Variante entsteht | ||
| * ist idealerweise ein einfaches Adjektiv | ||
| * ist idealerweise ein einfaches Adjektiv mit dem Prefix `is-` oder `has-` | ||
| * die Prefixe werden benötigt, damit ein Modifier einfach identifiziert werden kann und Zustände abgebildet werden können, für die Adjektive nicht ausreichend sind |
There was a problem hiding this comment.
Beispiele machen es klarer. Es gibt noch gar kein Beispiel für has.
|
Ich muss das dann nicht noch einmal reviewen, gerne einfach mergen. 🙂 |
SimonEggert
approved these changes
Nov 11, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Wir haben uns gestern im HTML/CSS-Austausch über die leicht unterschiedlichen Modifier Konzepte in ZASAF und den Whitelabel Components unterhalten.
In diesem Fall haben wir uns darauf geeinigt, dass wir das Konzept mit den Prefixen
isoderhassinnvoller finden.Folgende Gründe sprechen für die Umbenennung:
has-oderis-innerhalb von Dateien)is-bzw.has-submenuedkann es zu Unklarheiten führen, wenn man zusätzlich noch einesubmenuKomponente besitzt.Gegen die Umbenennung spricht, dass
is-undhas-keinen Vorteil für die individuelle Semantik eines Modifier enthalten.Ein Alternativvorschlag wäre, die Modifier per Zeichen zu trennen, z.B.
button | primaryo.button [ primary disabled ]. Damit würde man, neben der Nutzung einiger oben genannter Vorteile, auch dem Nachteil der Umbennenung entgegenwirken.