Skip to content

Conversation

nurpax
Copy link
Contributor

@nurpax nurpax commented Feb 28, 2019

Add a direct link to Python library documentation on readthedocs into
the first section of the README.md. This is the section that many
people see first when they google for pyimgui Python docs, so it's
helpful to link to library user docs right away.

I realize that the readthedocs link is already in the project
description but at least I missed that completely and had to scroll
many pages down to hit the readthedocs link. This is what many people
looking at the github project will be looking for.

Also drop "Cython-based" as IMO that doesn't help first time readers
of this README.md get an idea of what this project is about. IMO
"python bindings for ImGui" is more direct and informative.

Add a direct link to Python library documentation on readthedocs into
the first section of the README.md.  This is the section that many
people see first when they google for pyimgui Python docs, so it's
helpful to link to library user docs right away.

I realize that the readthedocs link is already in the project
description but at least I missed that completely and had to scroll
many pages down to hit the readthedocs link.  This is what many people
looking at the github project will be looking for.

Also drop "Cython-based" as IMO that doesn't help first time readers
of this README.md get an idea of what this project is about.  IMO
"python bindings for ImGui" is more direct and informative.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 46.49% when pulling 5953aa2 on nurpax:readme-docs-link into bc67eb8 on swistakm:master.

@swistakm swistakm merged commit a1789cc into pyimgui:master Mar 3, 2019
@swistakm
Copy link
Member

swistakm commented Mar 3, 2019

That makes sense. Thanks for contribution!

@nurpax nurpax deleted the readme-docs-link branch March 4, 2019 15:40
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