#22021 closed task (blessed) (fixed)
Integrate new media workflow into QuickPress
| Reported by: | koopersmith | Owned by: | azaozz |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | Media | Version: | 3.5 |
| Severity: | normal | Keywords: | needs-patch |
| Cc: | Focuses: |
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
@
14 years ago
- Resolution → fixed
- Status new → closed
[22072] is top-notch. Big step towards back compat.
#3
@
14 years ago
- Keywords needs-patch added
- Resolution fixed
- Status closed → reopened
Seems like it's broken again.
#5
@
14 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.
#7
follow-up:
↓ 9
@
14 years ago
- Resolution fixed
- Status closed → 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
@
14 years ago
Replying to koopersmith:
Nice catch. Keep forgetting thet focusin is normalized in jQuery (it's not supported in Firefox).
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
In [22072], by koopersmith:
Add QuickPress support for new media workflow.
media_button().media-uploadrequiremedia-views.see #21390, fixes #22021.