WordPress.org

Make WordPress Core

#24062 closed defect (bug) (fixed)

Force gallery state for gallery post format

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

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 months ago.
24062.2.diff (2.7 KB) - added by wonderboymusic 12 months ago.
24062.3.diff (507 bytes) - added by adamsilverstein 11 months ago.
reintroduce if ( workflow ) check

Download all attachments as: .zip

Change History (13)

lessbloat13 months ago

comment:1 lessbloat13 months 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.

wonderboymusic12 months ago

comment:2 wonderboymusic12 months ago

Patch refreshed / cleaned up against trunk

comment:3 markjaquith12 months ago

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

In 24087:

Force gallery state for gallery post format.

props lessbloat, wonderboymusic. fixes #24062.

comment:4 markjaquith12 months ago

In 24088:

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

see #24062

comment:5 follow-up: programmin11 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

Version 0, edited 11 months ago by programmin (next)

adamsilverstein11 months ago

reintroduce if ( workflow ) check

comment:6 in reply to: ↑ 5 adamsilverstein11 months 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

comment:7 ocean9011 months ago

  • Keywords needs-testing removed
  • Milestone 3.6 deleted
  • Resolution set to invalid
  • Status changed from reopened to closed

comment:8 SergeyBiryukov11 months ago

  • Milestone set to 3.6
  • Resolution invalid deleted
  • Status changed from closed to reopened
  • Version set to trunk

24062.3.diff still seems valid.

comment:9 wonderboymusic10 months ago

  • Keywords commit added

comment:10 markjaquith10 months ago

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

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.