[#11874] - Media Manager restyling
- Closed
- 1 Nov 2016
- Medium
- Build: staging
- # 11874
- Diff
- ciar4n:media-restyle
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Pending JTracker/HumanTestResults Human Test Results: 1 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
A potential restyling of the Media Manager.
Summary of Changes
Only Isis template.css is effected. Increased size of image preview with minimal increase in thumbnail footprint. Centered display of thumbnails across all image ratios and orientation.
Testing Instructions
Install the patch and browse to Media Manager. Full refresh probably needed after applying patch. For the moment does not effect any modals, only Content->Media
Before Patch
After Patch
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Templates (admin) Administration |
| Labels |
Added:
?
|
||
- I have tested successfully.
-
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11874.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11874.
@C-Lodder I agree. The single click delete button is unnecessary and unforgiven. Probably best to keep a preview option in there. I considered what you suggest via CSS however it meant editing the com_media view to append the checkbox with a label. Seems easily achievable however my PHP knowledge is very limited so I took the safer route and opted against.
@brianteeman Thank you... I missed that. Testing throws me an error (Fatal error: Call to undefined function finfo_open) but I'm guessing it supersedes anything I'm doing here. Closing.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-09-01 11:57:29 |
| Closed_By | ⇒ | ciar4n |
I would definitely submit your changes to that repo!!
On 1 September 2016 at 12:57, Ciaran Walsh notifications@github.com wrote:
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11874 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABPH8b_N1kTxoLxZ-MG3QN7OSJx1FmxJks5qlr2qgaJpZM4JyiEw
.
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
@brianteeman Just wondering if the improved media manager doesn't make it for 3.7, should I re-open this PR (or a version of it)?
| Status | Closed | ⇒ | New |
| Closed_Date | 2016-09-01 11:57:27 | ⇒ | |
| Closed_By | ciar4n | ⇒ |
sounds good
I have tested this item
It does what it says and from a usability perspective I prefer it to #12643
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/11874.
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-11-01 09:19:28 |
| Closed_By | ⇒ | ciar4n |
Nice. Out of all honesty though, It would be great to simplify the media manager so that it removes the need for a checkbox AND delete button for each image.
Something like this, so the page dedicates all space to the image and all focus goes on them. Just my opinion: