refactor about backstopjs HOT 5 CLOSED

I can tell from your wish list that we are on the same page. All this stuff makes total sense. Even without tests I am really happy to pull this stuff in -- it's worth the short term regression risk. I would also add that I've gotten a lot of feedback on adding a ready event listener to the config -- e.g. a developer could emit a 'myAppIsReady' event on the body or in the console and BackstopJS would wait to test until that event is fired. This would really be easy to implement in Casper.

From the feedback I am getting I know the changes you're proposing, in addition to the ready event would create a lot of happiness.

Please do it and request the pull -- let me know if you need any support!

from backstopjs.

if you do go ahead with the changes -- I would just suggest you set the default reference directory location to be in parallel with the backstop.json file. It would simplify the on-boarding process for new users.

from backstopjs.

Hey Peter, I just wanted to check in with you. There has been considerable work done on BackstopJS: updating to an all-NPM install, simplifying the config files, adding custom screenshot events and cleaning up some of the underlying Gulp code. One of the next items would be the customizable screenshot location -- this is one of the items you had proposed above.

I wanted to see if you had gotten anywhere with that or if you still had any intentions of doing so. Please let me know. Best regards. GS

from backstopjs.

I went a slightly different direction and built this: https://github.com/skyglobal/Sheut

relies on node-canvas and is pain having to install cario...

from backstopjs.

Looks like it's all server side -- that's cool. Alright then, best of luck with it. Take care. GS

from backstopjs.