the-spec icon indicating copy to clipboard operation
the-spec copied to clipboard

Config Option for Pattern State Lineage Ripple Effect

Open bradfrost opened this issue 7 years ago • 1 comments

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.

bradfrost avatar Oct 19 '17 15:10 bradfrost

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: 👎

bmuenzenmeyer avatar Oct 19 '17 15:10 bmuenzenmeyer