Make WordPress Core

Opened 6 years ago

Last modified 10 months ago

#44867 accepted enhancement

image editor: save modified image as a new copy

Reported by: drzraf's profile drzraf Owned by: joedolson's profile joedolson
Milestone: Future Release Priority: normal
Severity: normal Version: 5.2
Component: Media Keywords: has-patch
Focuses: administration Cc:

Description

WordPress allows multiple image sizes to be defined by a theme : they will affect how hard-crop/soft-crop is being used.

Sometimes no autocropping is desirable because editors want to care themselves about it.
For example blog posts having two images ratios (thumbnail for listing, 3:1 for wide-width header).
Giving the possibility to derive one image from the other is very interesting : it avoid uploading two different images but still gives editor ability to do fine-tuned cropping.

For this to work the image editor must be able to save a modified image as a COPY of the original image.

Would an implementation be accepted?

https://wordpress.org/ideas/topic/image-editing-option-to-save-a-copy

Change History (38)

#1 @drzraf
6 years ago

  • Keywords has-patch added

Closing image editor triggers a imageEdit.close() does this._view.back().
But there is no way that I know to refresh media editor (from this._view) afterwards, but that would be needed to show the newly created attachment.

Other elements open to discussion:

  • The image image suffix (-e<timestamp>) may not fit (it'd mess with images backups).
  • The title substitution (appending the hardcoded string -mediacopy)
  • The fact that upload directory is kept to the value of the original file (instead of putting <current-year>/<current-month>)
Last edited 6 years ago by drzraf (previous) (diff)

#2 @mukesh27
6 years ago

  • Focuses administration added

#3 @drzraf
6 years ago

image-edit.php: @nacin @adamsilverstein ?

#4 follow-up: @adamsilverstein
6 years ago

For this to work the image editor must be able to save a modified image as a COPY of the original image.

@drzraf can you explain what you are trying to do?

i don't think i understand why this is required - why you need a copy of the image (can you use the original itself)? can you do what you need with custom crop/sizes?

#5 in reply to: ↑ 4 @drzraf
6 years ago

Replying to adamsilverstein:

For this to work the image editor must be able to save a modified image as a COPY of the original image.

@drzraf can you explain what you are trying to do?

Editor wants to add a rectangular 3:1 image to a blog post. He uploads one adequate image as an image field.

Then he wants to also provide a square image as a (optional) thumbnail (be it for search-listing, OpenGraph image, ... ...):

  • Solution 1: Use the same header image as the thumbnail.
  • => Thumbnail is auto-cropped (center, North-East, ...) but this is probably useless because the (square) region of interest is not there and must be selected by hand.
  • Solution 2: Upload the image a second time and crop it by hand.
  • => ok, but could be better
  • Solution 3: Crop it locally using a desktop application, then upload
  • => ok, but could be better

If user crops the original image (rather than a copy), he looses the possibility to use the original rectangular one as a 3:1 header image.

I think it makes sense to allow editors to create multiple derivatives from one image without having the upload the original file multiple times.

#6 @adamsilverstein
6 years ago

Thanks for explaining. Typically you would have predefined crops/aspect rations in your theme for different image placements, however I can see a use case for the ability to duplicate an existing media item, giving you a copy you can then do what you want with. Would a "duplicate image" function in the medial library fit your needs?

This feature might make more sense as something a plugin can provide.

#7 @drzraf
6 years ago

Yes, a duplicate media would be fine.
That way editors could quickly define various flavors of one image, cropping the way they want in order to get extract the best portion according to their human feeling.

I'd personally prefer to see a "duplicate" button in the media editor because that's the place/moment people would look for it so that would improve efficiency:
Just click "duplicate" then immediately start editing/cropping the copy.

NB: I think a kind of medias relationship should be maintained somewhere (post meta?)

#8 @drzraf
6 years ago

Dear WP team,
could you advise how to move forward on this?

Version 0, edited 6 years ago by drzraf (next)

@drzraf
6 years ago

improvement: if cropping as a copy, correctly change corresponding metadata

#9 @pento
6 years ago

  • Version trunk deleted

#10 @drzraf
6 years ago

  • Version set to trunk

#11 @drzraf
5 years ago

Patch needs reroll for 4.3, but before doing that or engaging myself in indefinite rerolls, I'd really like to get more comments from upstream about this feature, implementation and inclusion.

#12 @joedolson
2 years ago

  • Milestone changed from Awaiting Review to 6.1

I think incorporating this into the image editor experience would help to increase the usability of the editor, since the user would more easily understand the consequences of their action: choosing between 'duplicate and create new' vs 'modify current' makes it clear what the work you're doing will impact.

I'll keep this open; but I'm going to work on integrating these changes with #50523.

Thanks for persisting on this, @drzraf!

This ticket was mentioned in Slack in #core-media by joedolson. View the logs.


2 years ago

#14 @joedolson
2 years ago

  • Owner set to joedolson
  • Status changed from new to accepted

This ticket was mentioned in Slack in #core-media by joedolson. View the logs.


2 years ago

This ticket was mentioned in Slack in #core-media by joedolson. View the logs.


2 years ago

#17 @joedolson
2 years ago

  • Milestone changed from 6.1 to Future Release

#18 @joedolson
22 months ago

  • Milestone changed from Future Release to 6.3

This ticket was mentioned in Slack in #core-media by antpb. View the logs.


18 months ago

#20 @joedolson
18 months ago

  • Keywords needs-refresh added

This is still on my radar, but I'm not going to work on it until after #50523 is finished. I don't want to incorporate more changes into that patch; it's pretty large already.

The hard part here is the one-line 'todo' statement in the existing patch, so while this has a patch, it's missing a crucial piece.

This ticket was mentioned in Slack in #core-media by antpb. View the logs.


18 months ago

#22 @antpb
18 months ago

  • Milestone changed from 6.3 to 6.4

@joedolson mentioned one pending todo in the patch to refresh after saving and indication when that happens.

Still needs a bit of work but unlikely to make it in the next few days. Moving to 6.4

This ticket was mentioned in Slack in #core-media by antpb. View the logs.


15 months ago

#24 @oglekler
15 months ago

  • Milestone changed from 6.4 to 6.5

There is no development in last 15 months, moving it into 6.5 instead of Future release for visibility.

#25 @drzraf
11 months ago

rerolled for WP 6.2

Could anyone take this over and make the necessary changes so that it could get in? I don't know how many rerolls/years are needed (on average) to get something in.

#26 @drzraf
11 months ago

  • Keywords needs-refresh removed

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


10 months ago

#28 @audrasjb
10 months ago

  • Milestone changed from 6.5 to Future Release

As per today's bug scrub:
This ticket is not ready yet as the patch needs to be tested before being committed.
Moving to Future Release.

#29 @joedolson
10 months ago

There's a big open todo in the patch, and that's the main thing preventing this from moving forward. The rerolls are appreciated, but ultimately we're going to need the patch to get finished before we can take the next steps. Hope to be able to figure that out in the next release.

Note: See TracTickets for help on using tickets.