Make WordPress Core

Opened 14 years ago

Closed 12 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:
Priority: normal Milestone:
Component: Media Version: 3.5
Severity: normal Keywords:
Cc: Focuses: ui

Description

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
14 years ago

  • Component UIMedia
  • Keywords ui-focus added

#2 @ocean90
13 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
13 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
13 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
12 years ago

  • Keywords needs-patch removed
  • Milestone Awaiting Review
  • Resolutionduplicate
  • Status newclosed

Duplicate of #27028.

Fixed in [29831]

Note: See TracTickets for help on using tickets.