Skip to content

Improved formatting#4682

Merged
hugovk merged 4 commits into
python-pillow:masterfrom
radarhere:formatting
Jun 12, 2020
Merged

Improved formatting#4682
hugovk merged 4 commits into
python-pillow:masterfrom
radarhere:formatting

Conversation

@radarhere

Copy link
Copy Markdown
Member

All changes should be obvious from the diff, except for the ImageGrab change, which fixes the formatting from this screenshot.

xdisplay


:param xdisplay: X11 Display address. Pass ``None`` to grab the default system screen.
Pass ``""`` to grab the default X11 screen on Windows or macOS.
:param xdisplay: X11 Display address. Pass ``None`` to grab the default system screen. Pass ``""`` to grab the default X11 screen on Windows or macOS.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

I don't quite understand the logic of why it was doing the odd formatting before and why this fixes it. Do you?

Comment thread src/PIL/ImageCms.py
@@ -192,9 +192,9 @@ class ImageCmsTransform(Image.ImagePointHandler):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this file, should we also give code formatting/linking to:

  • ImageCms.buildTransform() or ImageCms.buildTransformFromOpenProfiles())
  • If you want to modify im in-place
  • ImageCms.INTENT_PERCEPTUAL = 0 (DEFAULT)
    ImageCms.INTENT_RELATIVE_COLORIMETRIC = 1
    ImageCms.INTENT_SATURATION = 2
    ImageCms.INTENT_ABSOLUTE_COLORIMETRIC = 3
  • CmsProfile

Should PyCMSError be code formatted rather than bold?

https://pillow--4682.org.readthedocs.build/en/4682/reference/ImageCms.html

Comment thread src/PIL/ImageCms.py Outdated
radarhere and others added 3 commits June 12, 2020 07:51
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
@hugovk
hugovk merged commit 63ef8be into python-pillow:master Jun 12, 2020
@hugovk

hugovk commented Jun 12, 2020

Copy link
Copy Markdown
Member

Thanks!

@radarhere
radarhere deleted the formatting branch June 12, 2020 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants