Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
The latest plupload library provided a new widget with jQuery UI, which provides richer feature such as thumbnails view and drag and drop sorting.
Proposed resolution
Add new attributes on plupload element for choosing the widget to apply. The proposed new attribute name is #plupload_widget and its allowed values are "queue" and "ui". The default value is "queue".
Remaining tasks
The current patch is done on 7.x-1.x , we need to make it on 7.x-2.x as well.
jQuery 1.7 or above is required for the new feature. We need to implement dependency checking and display message when the new widget is loaded.
Comment | File | Size | Author |
---|---|---|---|
#3 | Screen Shot 2017-09-08 at 09.21.57.png | 84.79 KB | DrupalGideon |
#3 | Screen Shot 2017-09-08 at 09.21.44.png | 74.56 KB | DrupalGideon |
#3 | Screen Shot 2017-09-08 at 09.21.15.png | 83.37 KB | DrupalGideon |
#2 | plupload-make_compatible_with_v219-2731695-2-D7.patch | 21.63 KB | jimmyko |
Comments
Comment #2
jimmyko CreditAttribution: jimmyko as a volunteer commentedThis patch is same as the one in this comment
Comment #3
DrupalGideonI just added this patch (it wouldn't patch properly so I had to do some manual work) and I'm using the 2.x-dev branch of the module. It seems to work fine - apart from one issue which is styling.
If I set the
views
settings for#plupload_settings
so thelist
isTRUE
and thethumbs
isFALSE
, then the header in my admin theme (which uses CSS reset) is really short. If I then set another setting e.g.max_file_count
and then exceed the file count, the error that appears where the header is, is cut short.This could be avoided if the
.plupload_header_title
and.plupload_header_text
were not set todisplay: none;
.Screen grabs to show what I mean are attached.
If this could be adjusted, I see no reason why this patch can't go into the module as it's really handy.
Comment #4
DrupalGideonCorrection to above - it does not work with FileField Sources Plupload. So still needs work.
My FileField Sources widget defaults to the queue version by the looks of it, but it breaks. Even switching to the UI version to test (by quickly patching FileField Sources Plupload) still doesn't work either.
So more work needed.
Comment #5
jimmyko CreditAttribution: jimmyko as a volunteer commented@DrupalGideon
Sorry for this late reply. I will try to make a patch for 7.x-2.x branch and fix the issue asap as I have some spare time recently :)