#3191 closed enhancement (fixed)
Inline Uploading interface improvements
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 2.1 |
| Component: | Administration | Version: | 2.1 |
| Severity: | normal | Keywords: | JS |
| Cc: |
Description
Initial run.
- Accesss uploaded files through Manage -> Uploads.
- svn mv wp-admin/inline-uploading.php wp-admin/upload.php.
- Clearer "Send to editor" options.
- Fully pluggable tabs.
- Cacheable JS and CSS.
- Can edit file titles and descriptions.
- JS has random version string for debugging.
To Do:
- Split tabs into This Post / Images / Files ?
- UI/Layout improvements?
Attachments (13)
Change History (27)
comment:1
mdawaffe
— 7 years ago
- Owner changed from anonymous to mdawaffe
- Status changed from new to assigned
comment:2
markjaquith
— 7 years ago
comment:3
mdawaffe
— 7 years ago
I shouldn't have svn moved. Didn't think of that, sorry.
svn mv wp-admin/inline-uploading.php wp-admin/upload.php on your local copy first, then try applying the patch. Hopefully that will work.
Further To Do:
- pagination. #3159 would help.
comment:4
markjaquith
— 7 years ago
Nah, that doesn't work, because the patch tries to put code into both inline-uploading.php and upload.php
comment:7
Nazgul
— 7 years ago
Does this also fix #3173? I don't have Opera available here to test it.
comment:8
foolswisdom
— 7 years ago
mdawaffe updated ticket:3173 suggesting it does. Thanks Nazgul.
comment:10
mdawaffe
— 7 years ago
3191d.diff
- Don't include admin-header.php for inline.
- Some bugs fixed when going between JS and no JS.
- Thumbnail list items sohuld link to edit in Manage->Uploads.
comment:11
ryan
— 7 years ago
comment:12
mdawaffe
— 7 years ago
Do nothing with 3191e.diff
Commit 3191e.2.diff
comment:13
Nazgul
— 7 years ago
mdawaffe, could you take a look at #899 and see if it can be combined with this one?
(and why are you attaching new patches to a closed ticket? shouldn't it be reopened then?)
comment:14
mdawaffe
— 7 years ago
Nazgul - no good reason to keep flooding this closed ticket :)
And I've closed #899
Patch isn't applying correctly...
/wp-admin/upload.php doesn't exist in trunk...