patternlab-node icon indicating copy to clipboard operation
patternlab-node copied to clipboard

View Pattern Info toggles broken when using Incremental Builds

Open dnwhte opened this issue 7 years ago • 5 comments

The view pattern info toggle is broken when using incremental builds. Saving a watched pattern fixes it for that individual pattern.

version: v3.0.0-alpha.5 cleanPublic: false (in config)

DevTools error: plerror

dnwhte avatar Dec 06 '17 20:12 dnwhte

Hi @danwhite85

First off, thanks so much for giving the alpha a shot! It's quite a bit of a shake up, and admittedly a WIP in places.

I'll take a look at this as soon as available.

bmuenzenmeyer avatar Dec 07 '17 13:12 bmuenzenmeyer

After some more testing, it doesn't seem to be directly related to incremental builds.

The issue is that pl-pattern-data for patterns does not get populated in styleguide.html if the public folder exists on serve.

dnwhte avatar Dec 07 '17 19:12 dnwhte

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Mar 17 '18 18:03 stale[bot]

I can confirm that there's definitely an issue with the Pattern "Extras" Toggle in the latest code up on master. Still digging into this but @danwhite85's comment about missing data gives me a couple ideas to try out. Hang tight!

sghoweri avatar Jul 22 '18 11:07 sghoweri

To add to this one, when using patternlab.build() directly, if you pass in cleanPublic: true patternData is populated and the extras toggle works correctly (because pl-js-pattern-data div is populated with the patternData variable in patternSection.mustache).

If you pass in cleanPublic: false then patternData is not populated which means the patternSection.mustache doesn't render the patternData variable.

This does effectively mean you can't use incremental builds unless you don't really care about seeing the extras info.

zhawkins avatar Sep 12 '19 18:09 zhawkins