pattern-lab / patternlab-node

The Node version of Pattern Lab
http://patternlab.io
MIT License
2.05k stars 405 forks source link

Reinvestigate page-follow #103

Closed bmuenzenmeyer closed 9 years ago

bmuenzenmeyer commented 9 years ago

Resurrect https://github.com/pattern-lab/patternlab-node/pull/86 and get this working some day

dmolsen commented 9 years ago

BrowserSync?

bmuenzenmeyer commented 9 years ago

That's what I integrated for a moment but the iframe nab messaging was not keeping up. I think it's possible but I didn't want it holding up a minor rev. I should put back into a branch for experimentation.

On Sunday, March 1, 2015, Dave Olsen notifications@github.com wrote:

BrowserSync http://www.browsersync.io?

— Reply to this email directly or view it on GitHub https://github.com/pattern-lab/patternlab-node/issues/103#issuecomment-76595768 .

ivanmayes commented 9 years ago

Just FYI, browsersync hit 2.0 and now has a seperate UI for syncing devices, getting error reporting in one place, etc. I've used it with a fork of patternlab and can sync devices to a page via the UI History Panel, but the Patternlab nav still doesn't trigger it. Will take some time at some point to find the event its listening for and maybe we can just trigger that event when the ui changes.

bmuenzenmeyer commented 9 years ago

That's exactly where I got and then stopped. I'm hoping the fix is simple, only subtle. Thanks for the info on the new 2.0

On Mon, Mar 2, 2015 at 8:23 AM, Ivan Mayes notifications@github.com wrote:

Just FYI, browsersync hit 2.0 and now has a seperate UI for syncing devices, getting error reporting in one place, etc. I've used it with a fork of patternlab and can sync devices to a page via the UI History Panel, but the Patternlab nav still doesn't trigger it. Will take some time at some point to find the event its listening for and maybe we can just trigger that event when the ui changes.

— Reply to this email directly or view it on GitHub https://github.com/pattern-lab/patternlab-node/issues/103#issuecomment-76719364 .