Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Return to typewriter mode suggestion #47

Conversation

Not-Abram
Copy link
Contributor

I was assigned to play through the game by my professor. When I got to the end of the tutorial world, I left the game in the editor mode, and was confused when I opened up the next world. I believe it would be helpful to add a note in the intro of the first level in the addition world reminding users to switch back to the typewriter mode. My professor also added a change to use consistent language for referencing the typewriter mode.

@joneugster
Copy link
Collaborator

Recently we changed it so that Editor mode is persistent after completing a level, as some people preferred to play the entire game in editor mode.

If somebody has an opinion that it should be otherwise, i.e. you start in typewriter in each level, I'm open to have an issue at lean4game talking about that. We need one of the two behaviours, but I don't have a clear opinion what's better for the majority of users :)

@joneugster joneugster requested a review from kbuzzard January 26, 2024 11:01
@StevenClontz
Copy link
Contributor

StevenClontz commented Jan 31, 2024

My hot take is that the current behavior of persisting (or having the ability to persist) the typewriter/editor preference is a good idea; I can see folks playing the game that want to get into "real" Lean programming sooner than later would prefer the interface closer to actually writing Lean.

I suppose a checkbox could appear in the corner of the editor mode that says "stay in editor mode" - then if that's ticked, you stay in editor mode from level to level; if it's unticked then you revert back to typerwriter mode when changing levels.

@joneugster joneugster merged commit d24f07d into leanprover-community:main Feb 2, 2024
1 check passed
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.

3 participants