Opened 10 years ago
Closed 9 years ago
#32779 closed defect (bug) (fixed)
Changes to captions aren't synced to the editor's gallery view
Reported by: | ryan | Owned by: | iseulde |
---|---|---|---|
Milestone: | 4.3 | Priority: | high |
Severity: | normal | Version: | 4.3 |
Component: | Editor | Keywords: | make-flow has-patch |
Focuses: | ui | Cc: |
Description
This was previously fixed in #31329. The bug is back.
To reproduce, add a gallery without captions, click edit, add a caption to a photo, update and nothing changes. Updating the post will refresh the gallery and then the caption is visible.
Attachments (4)
Change History (16)
This ticket was mentioned in Slack in #core-editor by boren. View the logs.
10 years ago
#3
@
9 years ago
@iseulde: if you wouldn't mind reviewing this (32779.patch) I would appreciate it. I'm not sure if overloading getContent()
to do an initial render is the idiomatic way, but I came to that when attempting to prevent duplicating the code.
Also, sorry I stole your ticket - I'll look on the assigned-list in the future
#5
@
9 years ago
Ideally any change to captions in the media library should update all the gallery views in the editor. They're not stored in the shortcode, so we can't detect a change by looking at that. The best thing to do, I think, is when a gallery view is updated, destroy all instances and render everything again.
This ticket was mentioned in Slack in #core by jorbin. View the logs.
9 years ago
#7
@
9 years ago
- Keywords needs-patch added; has-patch removed
This needs a patch that works as @iseulde outlines. This is a regression so we should get it fixed before release.
Move rendering into
getContent()