Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(458)

Issue 6406045: code review 6406045: gotour: achievement system

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 6 months ago by tux21b
Modified:
12 years, 7 months ago
Reviewers:
adg, golang-dev, bradfitz
Visibility:
Public.

Description

gotour: achievement system Added a simple achievement system to keep new gophers motivated. The achievement bubbles are displayed on the bottom right corner and multiple bubbles will stack automatically on top of each other. You can test this functionality by running the program on slide #14 with an empty localStorage. This CL only introduces 6 example achievements, but the plan is to add quite a lot of them. Feel free to propose additional achievements!

Patch Set 1 #

Patch Set 2 : diff -r ca1c030f0dfb https://code.google.com/p/go-tour #

Patch Set 3 : diff -r ca1c030f0dfb https://code.google.com/p/go-tour #

Unified diffs Side-by-side diffs Delta from patch set Stats (+180 lines, -5 lines) Patch
M gotour/goplay.go View 1 3 chunks +76 lines, -2 lines 0 comments Download
M static/index.html View 1 1 chunk +4 lines, -0 lines 0 comments Download
M static/playground.js View 1 2 chunks +6 lines, -1 line 0 comments Download
M static/tour.css View 1 1 chunk +43 lines, -0 lines 0 comments Download
M static/tour.js View 1 4 chunks +51 lines, -2 lines 0 comments Download

Messages

Total messages: 3
tux21b
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go-tour
13 years, 6 months ago (2012-07-14 20:50:44 UTC) #1
adg
Hey, I did a similar thing here: http://codereview.appspot.com/5652075/ It has a more complex way to ...
13 years, 4 months ago (2012-08-27 05:51:11 UTC) #2
bradfitz
12 years, 7 months ago (2013-06-05 20:52:35 UTC) #3
R=close

I assume this no longer applies.  I think adg still wants to do something like
this eventually, so maybe this could be resurrected.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b