Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#28933 closed defect (bug) (fixed)

Adding New Media 'shivers' and loses focus when tabbing/navigating between fields

Reported by: Ipstenu Owned by: ericlewis
Milestone: 4.0 Priority: normal
Severity: normal Version: 4.0
Component: Media Keywords:
Focuses: Cc:


When adding new media to a post, the media editor has some serious jiggle issues and focus loss.

To Reproduce:

1) Make a new post
2) Upload a NEW image to that post
3) Change the image title
4) Hit tab OR click to the next field

The screen will 'jerk' as it's reloading something (don't know what) and then there will be nothing selected for focus.

Video illustrations:
https://cloudup.com/cLl7hyNWKcH (MOV)
https://cloudup.com/cMZDBUUQHYH (MP4)
https://cloudup.com/creXb9gahEv (M4V - slightly different example)

Related to #25100 and #23560

Change History (7)

#1 @SergeyBiryukov
7 years ago

  • Milestone changed from Awaiting Review to 4.0

#2 @ericlewis
7 years ago

Introduced in r28993, in media.view.Attachment:

this.model.on( 'change', this.render, this );

I did this to allow data to be edited in the Media Grid Attachment Details modal and then auto-rendered in the grid.

Should figure out which view is performing the editing, and noop re-rendering for that.

#3 @Ipstenu
7 years ago

Current status.

Not 'shivering' but when I navigate between fields, the focus snaps back to the URL every time.

This happens on all images in the in-post image modal box, new and old.

This ticket was mentioned in IRC in #wordpress-dev by DrewAPicture. View the logs.

7 years ago

#5 @DrewAPicture
7 years ago

  • Keywords dev-feedback added
  • Owner set to ericlewis
  • Status changed from new to reviewing

@ericlewis: Any update on reproducing this issue and/or finding a fix?

#6 @Ipstenu
7 years ago

It appears to be working now.

#7 @wonderboymusic
7 years ago

  • Keywords dev-feedback removed
  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.