Skip to content

Conversation

potocpav
Copy link
Contributor

@potocpav potocpav commented Oct 30, 2019

This PR adds several functions from the raw drawing API: AddCircle, AddCircleFilled, AddText, AddImage, and AddPolyline. All of these are members of _DrawList, which I added to documentation.

Each of the newly-implemented functions have visual examples, and I tried my best to document them similarly to other functions in PyImGui. I didn't know if I should include the visual examples themselves, so I did.

This PR is a continuation of #103, but I added visual examples and limited the focus on _DrawList functions. It partly intersects with #123, but with (hopefully) better documentation. It also (partly?) solves #49.

@coveralls
Copy link

coveralls commented Oct 30, 2019

Coverage Status

Coverage increased (+0.5%) to 54.01% when pulling b8f0ed1 on potocpav:draw into c163865 on swistakm:master.

@potocpav
Copy link
Contributor Author

The deletions in imgui/core.pyx are just because I reordered the drawing functions to match the order in imgui

@swistakm
Copy link
Member

This is rad! Thank you for the contribution.

Sorry it took me so long to review.

@swistakm swistakm merged commit 2dfd461 into pyimgui:master Nov 21, 2019
@swistakm swistakm added the release pending Merged but still needs official release label Nov 21, 2019
@potocpav
Copy link
Contributor Author

Great that it got merged! Together with the other recent updates, this will enable me to throw away my fork and publish a project on pypi. Thanks :)

@potocpav potocpav deleted the draw branch November 22, 2019 10:27
@swistakm
Copy link
Member

Great to hear that. As soon as the build completes I will publish new version.

@swistakm
Copy link
Member

This has been released in 1.1.0 and is available for download from PyPI.

@swistakm swistakm removed the release pending Merged but still needs official release label Nov 22, 2019
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