search

GritLearning / grit

Grit Learning app
5 stars 1 forks source link

clicking on a locked level that isn't the "next" level should not do anything #43

Closed daveob closed 10 years ago

daveob commented 11 years ago

Currently it launches the quiz, but the quiz should only be launched by clicking on the "next" level.

ghost commented 11 years ago

I noticed this as well but does it matter? If they try to click on a locked level they'll have to write the test and if they can't pass it they'll get sent back. Is it deterrent issue or a performance issue?

On Tue, Oct 22, 2013 at 4:14 PM, daveob notifications@github.com wrote:

Currently it launches the quiz, but the quiz should only be launched by clicking on the "next" level.

— Reply to this email directly or view it on GitHubhttps://github.com/GritLearning/grit/issues/43 .

daveob commented 10 years ago

Minor bug. By making the upper levels "dead" to clicking, it makes it clear that the quiz is only for the "next" level. Otherwise, it can be confusing - you click on level 4, a quiz comes up, and it unlocks level 2.

daveob commented 10 years ago

Ideally, only the quiz icon would be clickable, not even the rest of the "next" level.

This reinforces the idea that you can only get to the dimmed apps if you do the quiz first.

eoinkelly commented 10 years ago

I'll make the quiz icon clickable and make the overlay not.

ghost commented 10 years ago

Can we make it so only the level above is clickable to sue t to the quiz and not every level?

On Friday, October 25, 2013, Eoin Kelly wrote:

I'll make the quiz icon clickable and make the overlay not.

— Reply to this email directly or view it on GitHubhttps://github.com/GritLearning/grit/issues/43#issuecomment-27044164 .

Sent from my iFown.

daveob commented 10 years ago

Yep, that's the idea.

daveob commented 10 years ago

Orsome.

On 1 November 2013 11:24, Eoin Kelly notifications@github.com wrote:

Closed #43 https://github.com/GritLearning/grit/issues/43 via d8212bbhttps://github.com/GritLearning/grit/commit/d8212bbcfba021cf925acdae4aad8f0c874c3f6e .

— Reply to this email directly or view it on GitHubhttps://github.com/GritLearning/grit/issues/43 .