How do I close the tutorial? about meteor-tutorials

[geekyme]

I've done it this way at the moment:

Session.set('tutorialEnabled', true);
  Template.hello.helpers({
    tutorialEnabled: function(){
      return Session.get('tutorialEnabled')
    }
  });
Template.hello.options = {
    steps: tutorialSteps,
    onFinish: function(){
      Session.set('tutorialEnabled', false)
    }
  }

looks hacky to me. any suggestion? the demo is here > http://testtut.meteor.com/

[mizzao]

Hi @geekyme,

Thanks for reminding me about not publishing the version with the finish button. I put it on Meteorite and updated the documentation as well.

What you did doesn't seem too hacky to me. If the tutorial is on top of an expensively rendered page, you will probably want to make sure that the page doesn't get re-rendered when the tutorial disappears (will be a non-issue with Meteor UI.) What you did also works if you want the tutorial to be turned back on.

Your demo is very helpful and I have linked to it. Would you mind improving a bit i.e. by adding a container around the content so that it is centered, and making things display as block so it's more obvious that you can highlight specific regions?

[geekyme]

Ok @mizzao I've adjusted the demo and also pushed my code here >> https://github.com/geekyme/testtut

Hope its good. Thanks :)

[mizzao]

@geekyme Wow, that is nice! Thank you very much for putting that together!

[mizzao]

Hey @geekyme, I'm thinking of merging your demo app as a subdirectory of the tutorials app, as I have done with my other meteor packages (user-status, autocomplete, and sharejs) and which has helped greatly for testing new features.

Once I do that, you can fork this project directly and edit both the source and the demo app, and you can delete that repo. Having everything in one repo will make it easier for people to fork and try out modifications. Does this sound good to you?

[geekyme]

yeah just do that. I think i have the repository here > https://github.com/geekyme/testtut

How do i merge my repo into yours?

[mizzao]

I've merged your repo as a subdirectory of tutorials using the method in http://stackoverflow.com/a/614254/586086 (2ca655d). You can delete your repo now, if you want. To make future changes you should just fork the tutorials repo and send a PR - or I can just give you commit access, if you want.

I'm in the process of making this work for Blaze and will update your example as well if any changes are needed.

[geekyme]

Ok cool thanks Andrew !

Sent from my iPhone

On 2 Apr, 2014, at 6:00 am, "Andrew Mao" [email protected] wrote:

I've merged your repo as a subdirectory of tutorials using the method in http://stackoverflow.com/a/614254/586086. You can delete your repo now, if you want. To make future changes you should just fork the tutorials repo and send a PR - or I can just give you commit access, if you want.

I'm in the process of making this work for Blaze and will update your example as well if any changes are needed.

—
Reply to this email directly or view it on GitHub.

[mizzao]

This works for Blaze now, and I've updated your tutorial accordingly. I've also added in an example of the new EventEmitter functionality.