the-spec
the-spec copied to clipboard
Config Option for Pattern State Lineage Ripple Effect
Per discussion over here: https://github.com/pattern-lab/patternlab-php-core/pull/139#issuecomment-337945453
Having pattern states affect larger components is typically a good idea. If the primary nav isn't done, the header isn't done, which means the homepage isn't done. It's good to visually see that.
But I can also see how people might only want to see a visual for patterns they've explicitly set statuses for, and not have the ripple effect happen. Perhaps there could be a config option that gives people the choice?
@EvanLovely mentioned a boolean flag like stateSetByLineage
could be helpful for teams to control whether the ripple effect (current behavior) happens or not.
My gut tells me this is low priority, but it's worth flagging it here.
Having pattern states affect larger components is typically a good idea. If the primary nav isn't done, the header isn't done, which means the homepage isn't done. It's good to visually see that.
It's not just visually important, it's vitally important if users rely on state to determine readiness for production. "Done" can mean that it passes rigorous QA, a11y, documentation, and dev standards.
I strongly believe making this a configurable option dilutes the pattern state and pattern lineage feature. If a user wants to designate a pattern with an arbitrary label that does not affect patterns that may consume it, use a tag
.
My vote: 👎