WordPress.org

Make WordPress Core

Opened 16 months ago

Last modified 3 months ago

#44790 new defect (bug)

Attachment Overlay does not trigger spinner for compat fields

Reported by: hjanuschka Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Media Keywords: has-patch needs-testing needs-refresh
Focuses: ui, javascript, administration Cc:
PR Number:

Description

we are using ACF and some other custom fields in the attachment library, when we change values of those "compat" fields - the spinner/loader does not show up, which leads to the situation that the editor/user, may think everything is saved, and clicks assign/close to the overlay, and behind the overlay he/she may click on publish, which in some situations cancels the XHR requests and leads to data loss.

i have prepared a patch, not 100% sure if it is in the right place, let me know if you'd want me to adopt/change anything.

Attachments (2)

attachment_compat_indicator.44790.diff (1.3 KB) - added by hjanuschka 16 months ago.
attachment_compat_indicator.44790.1.diff (1.3 KB) - added by hjanuschka 16 months ago.
fix spaces

Download all attachments as: .zip

Change History (7)

#1 @hjanuschka
16 months ago

for whatever reason, the this.controller.trigger( 'attachment:compat:ready' and this.controller.trigger( 'attachment:compat:waiting' did not reach the this.listenTo( this.controller, 'attachment:compat:waiting attachment:compat:ready', this.updateSave );

those i change dit to the .on() variation, i found somwhere else in the js codebase

#2 @pento
11 months ago

  • Version trunk deleted

#3 @joemcgill
8 months ago

  • Keywords needs-testing added
  • Milestone changed from Awaiting Review to 5.3

Hi @hjanuschka. Thanks for the report and for providing a patch. I'm going to move to the 5.3 milestone for testing/consideration.

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


3 months ago

#5 @davidbaumwald
3 months ago

  • Keywords needs-refresh added
  • Milestone changed from 5.3 to Future Release

The latest patch fails against trunk, so it needs a refresh. Also, this would need testing and review. With Beta 1 of version 5.3 only a few days away, this is being moved to Future Release. If this ticket can be resolved in time for 5.3, feel free to re-milestone.

Note: See TracTickets for help on using tickets.