-
Notifications
You must be signed in to change notification settings - Fork 0
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
Made a start with the tutorial #29
Conversation
docs/tutorial.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ziet er duidelijk uit, de opbouw is ook heel goed te volgen. Misschien dat het lastig is om te volgen dat het script uitbouwt. Dat kunnen we bijvoorbeeld oplossen door 1) de import statements te laten staan en daadwerkelijk het script in de tutorial ook uit te breiden, of 2) het script een naam te geven met title="simple_gui.py" daarmee worden de losse pythonblokken wat meer aan elkaar gekoppeld.
Eigenlijk is de import-statements stap 1, maar de drie stappen zijn nu ook wel heel mooi. Is het nog een idee om de drie stappen op aparte pagina's te zetten?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bedankt voor de feedback! Optie 2 die je hebt gegeven klinkt goed, dat ga ik proberen toe te passen. Ook een goed idee om de stappen op aparte pagina's te zetten, ik ga kijken of dit beter is.
docs/images/empty_window.png
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heel goed om te zien hoe het lege scherm eruit ziet. Dit is alleen het scherm in Windows, misschien moeten we dat er nog ergens bij vermelden. Of zoals bij bijvoorbeeld de tutorial van Beeware (https://docs.beeware.org/en/latest/tutorial/tutorial-0.html) met een tabje laten zien wat voor windows en hoe het er voor MacOS uitziet (heeft geen prioriteit)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oeh, dat zou wel mooi zijn inderdaad! Laten we later kijken of hier nog tijd voor is.
Ik heb de secties van de tutorial op aparte pages gezet en heb het script een title gegeven (title="simple_gui.py"). |
Ik heb een opzet gemaakt van de tutorial page op deze branch ("tutorial"). @AnneliesVlaar zou jij hier naar willen kijken en feedback willen geven? Bedankt!