Get rid of references to walkthrough.jpg

[jdoliner]

Currently our html files contain references to the old walkthrough images which don't actually exist on the server. There's no exposed links but this does lead to error messages if someone crawls the site. Generally I think it's poor form the have links to things that don't exists even if they're concealed.

A much lesser issue is that we get the same thing robots.txt ... I actually don't even consider that to be a real issue.

[coffeemug]

Or we could actually add a walkthrough, not that the UI is complete.

[neumino]

Should we get rid of the current (dead) walkthrough code?

I don't think the web UI is really hard to understand so I'm not sure we need one, but if we have to do a walkthrough, I think an overlay would be better than a slideshow.

[jdoliner]

I think honestly if we need a walkthrough to explain a UI we're doing
something wrong. And people seem to get and like the UI without one so I my
vote is to just nix this idea altogether.

On Fri, Jan 4, 2013 at 6:10 PM, Michel notifications@github.com wrote:

Should we get rid of the current (dead) walkthrough code?

I don't think the web UI is really hard to understand so I'm not sure we
need one, but if we have to do a walkthrough, I think an overlay would be
better than a slideshow.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/68#issuecomment-11907784.

[coffeemug]

I also vote to get rid of the code, for the following reasons:

• I don't think a walkthrough is a bad idea per se, a really nice UI + a simple introductory walkthrough can still be really nice. But I do think that to maintain the level of polish we have now, we'd have to do a well thought-out interactive mini-tutorial, not a slideshow, and the code we have now is useless for that.
• Considering how much people like the UI, and all the other stuff we'd like to do, it's unlikely this project will get done any time soon. So the code will bitrot even further.
• It's frankly not very difficult to put back in if we ever need it.

[neumino]

Done in commit 807bdbe
Code review 159.

I'll close when the review will be completed and the code merged in next.

[neumino]

Merged in e228168
Code review 159