Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#24062 closed defect (bug) (fixed)

Force gallery state for gallery post format

Reported by: lessbloat Owned by: markjaquith
Priority: normal Milestone: 3.6
Component: Post Formats Version: 3.6
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

One of the items uncovered with the last round of usability tests was that users don't know that they are not adding a gallery.

When the gallery post format is selected, and the user clicks the "Add Media" button, we should take them to the gallery section of the media modal.

Attachments (3)

24062.diff (2.7 KB ) - added by lessbloat 13 years ago.
24062.2.diff (2.7 KB ) - added by wonderboymusic 13 years ago.
24062.3.diff (507 bytes ) - added by adamsilverstein 13 years ago.
reintroduce if ( workflow ) check

Download all attachments as: .zip

Change History (13)

@lessbloat
13 years ago

#1 @lessbloat
13 years ago

24062.diff​ is a first attempt at making this happen. I'll include this patch in the next round of usability tests that I run.

#2 @wonderboymusic
13 years ago

Patch refreshed / cleaned up against trunk

#3 @markjaquith
13 years ago

  • Owner set to markjaquith
  • Resolutionfixed
  • Status newclosed

In 24087:

Force gallery state for gallery post format.

props lessbloat, wonderboymusic. fixes #24062.

#4 @markjaquith
13 years ago

In 24088:

Clean up gallery-mode toggling in post-formats.js and make sure it is set on first load.

see #24062

#5 follow-up: @programmin
13 years ago

  • Resolution fixed
  • Status closedreopened

Are you certain it is necessary to remove the

if ( workflow )
	return workflow

from the add function? Without this, current modifications that depend on getting a reference from the wp.media.editor.add function will break the uploading functionality, since other calls to add will create it again.

I have detailed the problem in this posting, if this can't be fixed I hope we can document a workaround.

http://lists.automattic.com/pipermail/wp-hackers/2013-May/046124.html

Last edited 13 years ago by SergeyBiryukov (previous) (diff)

@adamsilverstein
13 years ago

reintroduce if ( workflow ) check

#6 in reply to: ↑ 5 @adamsilverstein
13 years ago

Replying to programmin:

Are you certain it is necessary to remove the

if ( workflow )
	return workflow

from the add function? Without this, current modifications that depend on getting a reference from the wp.media.editor.add function will break the uploading functionality, since other calls to add will create it again.

I have detailed the problem in this posting, if this can't be fixed I hope we can document a workaround.

http://lists.automattic.com/pipermail/wp-hackers/2013-May/046124.html

24062.3.diff​ reintroduces the if ( workflow ) check & return

#7 @ocean90
13 years ago

  • Keywords needs-testing removed
  • Milestone 3.6
  • Resolutioninvalid
  • Status reopenedclosed

#8 @SergeyBiryukov
13 years ago

  • Milestone3.6
  • Resolution invalid
  • Status closedreopened
  • Versiontrunk

24062.3.diff still seems valid.

#9 @wonderboymusic
13 years ago

  • Keywords commit added

#10 @markjaquith
13 years ago

  • Resolutionfixed
  • Status reopenedclosed

In 24447:

Restore code that returns the existing media object if it already exists.

props programmin, adamsilverstein. fixes #24062

Note: See TracTickets for help on using tickets.