Skip to content
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 documentation link to PIL.ImageFont.Layout #21

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

nulano
Copy link

@nulano nulano commented Nov 30, 2023

For python-pillow#7578

Looking at the generated docs, I can see that the parameter type annotation Layout does not link to documentation for ImageFont.Layout because that class is not listed in the documentation.

Checking locally, I find that this commit fixes that.

@radarhere radarhere merged commit ea680d9 into radarhere:font Dec 1, 2023
@radarhere
Copy link
Owner

Thanks

@nulano nulano deleted the font branch December 1, 2023 01:48
radarhere added a commit that referenced this pull request Feb 12, 2025
* Simplify Python code by receiving tuple from C, as per python-pillow#8740

* Use default PyTypeObject value

* Removed AVIF_TRUE

* Width and height are already set on first frame

* Removed memset

* Depth is set by avifRGBImageSetDefaults

* Replace PyObject with int

* After a failed pixel allocation, destroy non-first frame

* Added error if avifImageCreateEmpty returns NULL

* Python images cannot have negative dimensions

* Test invalid canvas dimensions

* Use boolean format argument

* Handle avifDecoderCreate and avifEncoderCreate errors

* tileRowsLog2 and tileColsLog2 are ignored if autotiling is enabled

* Only define _add_codec_specific_options if it may be used

* Test non-string advanced value

* Simplified error handling in AvifEncoderNew

* Corrected heading

---------

Co-authored-by: Andrew Murray <radarhere@users.noreply.github.com>
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.

2 participants