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

Help button #8

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Help button #8

wants to merge 3 commits into from

Conversation

VitaJane
Copy link
Contributor

@VitaJane VitaJane commented Oct 8, 2024

Here's the final change we did. It just creates a help button, showing what the different functions are.

@GreyWyvern
Copy link
Owner

Thanks again for your pull requests!

As far as this one goes, there is definitely something to be said about making the current keyboard controls easier to understand at a glance. OTOH, I think this particular method, of popping up a whole new "window" with detailed text instructions goes quite a bit too far for inclusion in the basic script.

For one thing, it's a whole new "window", and secondly, it's all in English, which would need to have options for translations to be swapped in.

So while this is a neat idea, I would rather improve the way the current keyboard functions can be understood without adding extra text instructions. Cheers, though!

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