Make WordPress Core

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's profile ryan Owned by: iseulde's profile 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.

See https://make.wordpress.org/flow/2015/02/05/caption-changes-are-not-reflected-in-the-editor-macnchrome-4-2-alpha-31332/

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)

32779.patch (2.3 KB) - added by dmsnell 9 years ago.
Move rendering into getContent()
32779.2.patch (6.5 KB) - added by iseulde 9 years ago.
32779.3.patch (5.1 KB) - added by iseulde 9 years ago.
32779.4.patch (4.3 KB) - added by iseulde 9 years ago.

Download all attachments as: .zip

Change History (16)

This ticket was mentioned in Slack in #core-editor by boren. View the logs.


10 years ago

#2 @wonderboymusic
10 years ago

  • Owner set to iseulde
  • Status changed from new to assigned

@dmsnell
9 years ago

Move rendering into getContent()

#3 @dmsnell
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

#4 @dmsnell
9 years ago

  • Keywords has-patch dev-feedback added

#5 @iseulde
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 @jorbin
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.

#8 @obenland
9 years ago

@iseulde, do you think you could take a stab at finding a fix here?

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


9 years ago

@iseulde
9 years ago

#10 @iseulde
9 years ago

  • Keywords has-patch added; dev-feedback needs-patch removed

This ticket was mentioned in Slack in #core-editor by iseulde. View the logs.


9 years ago

@iseulde
9 years ago

@iseulde
9 years ago

#12 @iseulde
9 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 33338:

TinyMCE: views: make sure gallery captions update

Part props dmsnell.
Fixes #32779.

Note: See TracTickets for help on using tickets.