#28851 closed defect (bug) (fixed)
Media Grid: Improve uploader
Reported by: | ocean90 | Owned by: | |
---|---|---|---|
Milestone: | 4.0 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Media | Keywords: | |
Focuses: | ui, administration | Cc: |
Description (last modified by )
After clicking Add New the uploader flies in. The status of the button is unchanged, clicking it again will hide the uploader again. I think the button should hold the active state while the uploader is visible.
Correct me if I'm wrong but currently there is no other place where we let elements fly around. And we shouldn't start to this. I want the uploader immediately and don't want to wait for a second until the animation has finished. I prefer more something like we have on media-new.php or theme-install.php?upload, which is less distracting for uploading files.
We probably should also mention, that the whole screen is a dropzone for the uploader.
Related Issues:
- When you scroll to the end of the grid and click Add New you will get the uploader which overlays the the first half of the media. So it's possible that you won't see that you media has been uploaded because the new media is placed on top of the list, which I currently can't see. Feels wrong.
- Uploaded files will be selected, also when uploading multiple files, but multiple selection should be disabled until bulk edit is activated.
Attachments (6)
Change History (25)
#4
@
10 years ago
In 28851.3.diff:
- Parity of language and general styling on media-new.php.
- Adding the dashed line lets a user know that they can drag and drop media without reading any text.
- De-emphasize the select files button.
- Tighten up the close button placement.
- Remove a period at the end of "Maximum upload file size: 32 MB.", as that isn't really a sentence.
This ticket was mentioned in IRC in #wordpress-dev by ericandrewlewis. View the logs.
10 years ago
#7
@
10 years ago
Much better, separates No media found. Try a different search. message from uploader http://i.imgur.com/9xftDyt.png
#8
follow-up:
↓ 9
@
10 years ago
- Keywords needs-refresh added
<p>or</p>
has no i18n- Don't remove the fullstop in "Maximum upload file size: %s.", othwise we have two strings for the same (see /wp-admin/includes/media.php)
#9
in reply to:
↑ 8
@
10 years ago
- Keywords needs-refresh removed
Replying to ocean90:
- Don't remove the fullstop in "Maximum upload file size: %s.", othwise we have two strings for the same (see /wp-admin/includes/media.php)
Right, good idea wp-admin/includes/media.php#L1859
#14
follow-up:
↓ 15
@
10 years ago
- Keywords needs-patch added; has-patch removed
There is no visible error message when file size is above the maximum file size.
#15
in reply to:
↑ 14
@
10 years ago
- Keywords needs-patch removed
- Resolution set to fixed
- Status changed from new to closed
#16
follow-ups:
↓ 18
↓ 19
@
10 years ago
Remove margin: 0;
at wp-includes/css/media-views.css#L1273, because h3 is too near to the border http://i.imgur.com/wzbMy6S.png
#18
in reply to:
↑ 16
@
10 years ago
Replying to michalzuber:
Remove
margin: 0;
at wp-includes/css/media-views.css#L1273, because h3 is too near to the border http://i.imgur.com/wzbMy6S.png
Could somebody check that h3 fix 28851.5.diff ?
#19
in reply to:
↑ 16
@
10 years ago
Replying to michalzuber:
See ticket:28963:5.
attachment:28851.diff removes CSS transitions in lieu of some classic
display: none
. Also fixes #28840