-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
Fixing typos in turtle.rst #24385
New issue
8000
summary>
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
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
Fixing typos in turtle.rst #24385
Conversation
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.
LGTM, minor changes.
@@ -1894,7 +1894,7 @@ Input methods | |||
Pop up a dialog window for input of a number. title is the title of the | |||
dialog window, prompt is a text mostly describing what numerical information | |||
to input. default: default value, minval: minimum value for input, | |||
maxval: maximum value for input | |||
maxval: maximum value for input. |
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.
I think this is better rewritten as:
... *title* is the title of the dialog window, *prompt* is a text mostly describing what numerical information to input. *default* is a default value to fill the input with.
The number input must be in the range *minval* .. *maxval* (inclusive) if these are given. If outside of this range, a hint is issued and the dialog remains open for correction.
This PR is stale because it has been open for 30 days with no activity. |
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.
Since there is no activity to add the requested changes, and PR went stale, I will merge as-is for now.
@serverwentdown, you can open a PR for your suggestions if you wish.
Even more typos. (I'll keep updating the PR as I find more or open new ones)