Last modified: 2013-04-22 16:15:06 UTC
When visting http://piramido.wmflabs.org/wiki/Main_Page?tour=test, clicking the "X" to dismiss a tour should permanently dismiss the tour (at least during the current browser session). However, using ?tour=test and clicking X means that I retain the testwiki-mw-tour cookie and the beginning of the test tour appears incorrectly on every page load.
The first phase of my proposed fix is a change to our version of Guiders, which is awaiting review at https://gerrit.wikimedia.org/r/#/c/42599/ . After that is merged, I'll do a change to GuidedTour to bump the submodule version.
We probably should let this in now. But a more permanent change would be to make it so that whether endTour() or hideAll() is called is configurable depending on what action the user takes. It is conceivable in general (not for us) a user may want to just dismiss the box but not quit the tour completely (hence hideAll()).
Merged
The GuidedTour change is https://gerrit.wikimedia.org/r/#/c/42675/