Hook Docs (13): wp-admin/includes/image-edit.php

[theorboman]

Documenting all undocumented filter hooks in wp-admin/includes/image-edit.php

[theorboman]

Patch for inline docs

[markoheijnen]

Personally I rather have jpeg_quality description in /wp-includes/class-wp-image-editor.php. See #25721

[theorboman]

Updated patch for wp-admin/includes/image-edit.php. Added periods after variable descriptions as per standards. Also note that 'admin_memory_limit' is documented in wp-includes/class-wp-image-editor-imagick.php (see https://core.trac.wordpress.org/ticket/26177)

[DrewAPicture]

At a glance, the docs here look fairly complete, though the spacing is a little wonky. Perhaps you were using tabs instead of spaces or something. I'll see if I can get you a more thorough review today.

[theorboman]

Use spaces, not tabs to space out the inline comments.

[theorboman]

Here's another patch for this using spaces, not tabs for spacing out the docs. Let me know if this isn't the right way to do it!

[DrewAPicture]

3rd pass

[DrewAPicture]

26149.diff​ Should do it. Just needs a last review and recommendation.

[kpdesign]

Final pass

[kpdesign]

26149.3.diff​ contains a few tweaks to 26149.diff​.

This should be good to go. Recommend commit.

[DrewAPicture]

In 27357:

Inline documentation for hooks in wp-admin/includes/image-edit.php.

Props theorboman. Props kpdesign for the cleanup.
Fixes #26149.