#22021 closed task (blessed) (fixed)
Integrate new media workflow into QuickPress
Reported by: | koopersmith | Owned by: | azaozz |
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | normal | Version: | 3.5 |
Component: | Media | Keywords: | needs-patch |
Focuses: | Cc: |
Description
QuickPress calls media_buttons()
and has a textarea
, which means that the new media workflow should Just Work. Right now, it doesn't. Let's fix that.
Change History (9)
#2
@
12 years ago
- Resolution set to fixed
- Status changed from new to closed
[22072] is top-notch. Big step towards back compat.
#3
@
12 years ago
- Keywords needs-patch added
- Resolution fixed deleted
- Status changed from closed to reopened
Seems like it's broken again.
#5
@
12 years ago
There is a Quicktags instance for replying to comments on the dashboard and it uses wp_editor(). However QuickPress doesn't use wp_editor() so there's no JS to set wpActiveEditor
for it. Adding a little JS to fix that.
#6
@
12 years ago
- Owner set to azaozz
- Resolution set to fixed
- Status changed from reopened to closed
In 22517:
#7
follow-up:
↓ 9
@
12 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Setting wpActiveEditor
only on focus is insufficient — the "Add Media" button can be clicked without focusing either field. Also, we can simplify the selectors a bit, as well as leverage event delegation.
#9
in reply to:
↑ 7
@
12 years ago
Replying to koopersmith:
Nice catch. Keep forgetting thet focusin
is normalized in jQuery (it's not supported in Firefox).
In [22072], by koopersmith:
Add QuickPress support for new media workflow.
media_button()
.media-upload
requiremedia-views
.see #21390, fixes #22021.