WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#22628 closed defect (bug) (fixed)

Improve switching between browser and upload state

Reported by: koopersmith's profile koopersmith Owned by: ryan's profile ryan
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords: has-patch
Focuses: Cc:

Description

Currently, the attachments browser falls back to the upload state when there are no items in the collection. There are two major failings with the current implementation:

  1. When first navigating to the media library, you see a flash of the upload state before the attachments load.
  2. When a filter with no items is selected, we show an empty content area.

By moving the upload state and switching code into the browser view, we can fix both of these problems and eliminate complexity in the state.

Attachments (4)

22628.diff​ (4.2 KB) - added by koopersmith 12 years ago.
22628.filter.fix.png​ (75.4 KB) - added by koopersmith 12 years ago.
22628.2.diff​ (4.1 KB) - added by koopersmith 12 years ago.
22628.3.diff​ (4.3 KB) - added by koopersmith 12 years ago.

Download all attachments as: .zip

Change History (13)

@koopersmith
12 years ago

#1 @koopersmith
12 years ago

A nice touch after this would be to add spinners when attachments are loading.

Last edited 12 years ago by koopersmith (previous) (diff)

@koopersmith
12 years ago

#2 @koopersmith
12 years ago

This is what an empty filter looks like.

@koopersmith
12 years ago

#3 @koopersmith
12 years ago

Latest patch fixes vanishing view when switching back and forth.

#4 @ryan
12 years ago

Looks good.

#5 @ryan
12 years ago

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

This snuck in with [22919].

#6 @ryan
12 years ago

In 22923:

Revert accidental bits from r22919. see #22628

#7 @ryan
12 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This caused some problems with creating and editing galleries.

@koopersmith
12 years ago

#8 @koopersmith
12 years ago

Ah. Gallery problems were caused because they didn't have a content mode set. Fixed in latest patch.

#9 @ryan
12 years ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from reopened to closed

In 22945:

Improve switching between browser and upload state.

Props koopersmith
fixes #22628

Note: See TracTickets for help on using tickets.