Make WordPress Core

Opened 8 years ago

Closed 6 years ago

#22984 closed enhancement (duplicate)

Media Modal - Auto-save for compat-items doesn't trigger the UI to show the spinner or "Saved" indicators.

Reported by: eddiemoya Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords:
Focuses: ui Cc:


When the auto-save kicks in for the standard attachment editor fields, it triggers the spinner icon in the top right corner. This is followed by "Saved" which then disappears on its own. None of these UI indicators are triggered for custom input items added via the 'attachment_fields_to_edit hook.

Not sure if its was an oversight or simply not seen as important.

Change History (5)

#1 @sabreuse
8 years ago

  • Component changed from UI to Media
  • Keywords ui-focus added

#2 @ocean90
7 years ago

  • Keywords needs-patch added

Hello eddiemoya, sorry you never got a response.

You're right. There is no save indicator for compat-items. We should try to re-use the existing indicator. I'll take a stab on this.

#3 follow-up: @DrewAPicture
7 years ago

CMIIW, but it was my impression that the reason the spinner didn't activate is because the compat fields weren't being auto-saved like the default fields. The only way those fields would be saved would be on update.

#4 in reply to: ↑ 3 @ocean90
7 years ago

Replying to DrewAPicture:
The compat section has it's own view and is decoupled from the view with the spinner. So it's not that easy as I thought. The spinner should get it's own view.

But the fields are autosaved too, see trunk/src/wp-includes/js/media-views.js#L4954

#5 @wonderboymusic
6 years ago

  • Keywords needs-patch removed
  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Duplicate of #27028.

Fixed in [29831]

Note: See TracTickets for help on using tickets.