Opened 6 years ago
Last modified 10 months ago
#44867 accepted enhancement
image editor: save modified image as a new copy
Reported by: | drzraf | Owned by: | 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
Attachments (9)
Change History (38)
#1
@
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.
#4
follow-up:
↓ 5
@
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
@
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
@
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
@
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
@
6 years ago
Dear WP team,
could you advise how to move forward on this?
- As patch shows, there is no apparent way to implement this as a plugin.
- There is a missing bit about editor since, I don't know how to refresh the media browser listing (because a new file is added) when image editor is closed.
#11
@
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
@
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
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
This ticket was mentioned in Slack in #core-media by antpb. View the logs.
19 months ago
#20
@
19 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
@
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.
16 months ago
#24
@
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
@
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.
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
10 months ago
#28
@
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
@
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.
patch v1