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
Fix a few typos in the docs #2573
Conversation
doc/getting_started/modern-gl.rst
Outdated
|
||
|
||
Learning Modern 3D Graphics Programming | ||
--------------------------------------- | ||
|
||
This `book <https://paroj.github.io/gltut/>`_ is intended to teach you how | ||
`This book <https://paroj.github.io/gltut/>`_ is intended to teach you how |
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.
Is there a reason for this change?
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.
Seems a bit ugly to me to split the noun phrase. In this sense, 'book' could just be any book, the link could be even point to Wikipedia, the reader doesn't know.
The actual faux pas here is to conceal the name of the book.
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.
Good point! Do you want to put the real name of the online book there instead?
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.
Nice job. Thanks for fixing these things.
No description provided.