Context Navigation

← Previous Ticket
Next Ticket →

#38748 closed enhancement (fixed)

Improved documentation of image-edit.js

Reported by:	jjcomack	Owned by:	adamsilverstein
Priority:	normal	Milestone:	4.8
Component:	Media	Version:	4.7
Severity:	normal	Keywords:
Cc:		Focuses:	javascript, docs

Description

Please give props to andizer and rensw90.

Attachments (2)

image-edit.diff (19.2 KB ) - added by jjcomack 10 years ago.
38748.diff (19.2 KB ) - added by adamsilverstein 10 years ago.

Download all attachments as: .zip

Change History (8)

@jjcomack
10 years ago

Attachment image-edit.diff added

@adamsilverstein
10 years ago

Attachment 38748.diff added

#1 @adamsilverstein
10 years ago

@jjcomack Thanks for your work on this (and thanks to @andizer and @rensw90 as well)!

38748.diff includes some slight cleanup, added a few missing periods and fixed a few tiny grammar issues (a -> an).

#2 @jjcomack
10 years ago

@adamsilverstein Oh thanks. I overlooked those things!

This ticket was mentioned in Slack in #core by helen. View the logs.

10 years ago

#4 @helen
10 years ago

Milestone Awaiting Review → 4.8
Owner set to adamsilverstein
Status new → assigned
Type defect (bug) → enhancement

#5 @adamsilverstein
10 years ago

Resolution → fixed
Status assigned → closed

In 39493:

Docs: Add inline documentation for image-edit.js.

Adds JSDoc DocBlocks to the imageEdit methods.

Props: andizer, rensw90.
Fixes #38748.

#6 @SergeyBiryukov
10 years ago

In 39740:

Docs: Use 3-digit, x.x.x style semantic versioning for @since entries in wp-admin/js/image-edit.js.

See #38748.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#38748 closed enhancement (fixed)

Improved documentation of image-edit.js

Description

Attachments (2)

Change History (8)

@jjcomack 10 years ago

@adamsilverstein 10 years ago

#1 @adamsilverstein 10 years ago

#2 @jjcomack 10 years ago

This ticket was mentioned in ​Slack in #core by helen. ​View the logs. 10 years ago

#4 @helen 10 years ago

#5 @adamsilverstein 10 years ago

#6 @SergeyBiryukov 10 years ago