Skip to content

Conversation

@david-a-wheeler
Copy link
Contributor

Add delay for a hint. That way, instead of trying to use hints to write the answers, learners will be quietly encouraged to try to solve the problem themselves.

This adds a shorter delay (15 seconds) for each hint. If the user has seen a hint and hasn't changed anything, they can re-see the hint.

We earlier added a delay for "give up". This modifies the give up timer so that if they've asked for a hint, the give up timer starts from the time of the last hint. That way, people can give up, but they have to at least wait a little time.

Add delay for a hint. That way, instead of trying to use
hints to write the answers, learners will be quietly encouraged
to try to solve the problem themselves.

This adds a shorter delay (15 seconds) for each hint.
If the user has seen a hint and hasn't changed anything,
they can re-see the hint.

We earlier added a delay for "give up". This modifies the
give up timer so that if they've asked for a hint, the give up
timer starts from the time of the last hint. That way, people
*can* give up, but they have to at least wait a little time.

Signed-off-by: David A. Wheeler <[email protected]>
There's a standard solution for pluralization, but I don't think
we need it. Document the standard solution, in case we need to
use it later.

Signed-off-by: David A. Wheeler <[email protected]>
@david-a-wheeler
Copy link
Contributor Author

I talked with CRob, who thought this tweak to our labs was reasonable. I'm going to merge this now, to discourage the overuse of lab hints while retaining full hint functionality. We can make further changes later (e.g., tweaking the hint delay time).

@david-a-wheeler david-a-wheeler merged commit 133f2b7 into main Feb 6, 2025
4 checks passed
@david-a-wheeler david-a-wheeler deleted the hint_timer branch February 6, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants