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.
Display a representation of the selected style like D7 image styles.
Attached is an example.
Thoughts?
Comments
Comment #1
drupal a11y CreditAttribution: drupal a11y commentedGuess this Is the same as here: http://drupal.org/node/1407418
Comment #2
Jackinloadup CreditAttribution: Jackinloadup commentedMarked #1407418: Apply Image Styles to Thumbnail (after Image has been uploaded) as a duplicate.
In that issue it is expressed that this is "a really essential feature, especially if landscape pictures are used, the square preview is really bad to see what image was used." - mori
I moved this issue to 2.x b/c I don't believe any more features are being added to 1.x
Comment #3
John Pitcairn CreditAttribution: John Pitcairn commentedI really hope this would be an optional feature. I'd want to turn it off for every image field.
Comment #4
areikiera CreditAttribution: areikiera commentedJust checking in on the status of this feature request. Any updates?
Also, I agree with John Pitcairn, that it should be optional first, and with the ability to override by field.
Thanks!
Comment #5
Devin Carlson CreditAttribution: Devin Carlson commentedA patch to display a real preview of the file being embedded using WYSIWYG that updates via AJAX whenever a view mode is selected.
Pretty cool if I say so myself!
Comment #6
ParisLiakos CreditAttribution: ParisLiakos commentedthis is awesome!
I would just commit this, since code looks good too, but i want to make sure first with John Pitcairn and areikiera.
Could you please test Devin's patch and see if you still think this should be optional?
just commited this part:
http://drupalcode.org/project/media.git/commit/9ef998a
in case someone wants to add this through hook_form_alter().
Comment #7
ParisLiakos CreditAttribution: ParisLiakos commentedOh, i guess this needs to be rerolled now:/
Comment #8
Devin Carlson CreditAttribution: Devin Carlson commentedReroll of #5.
I'll probably add this to Media WYSIWYG View Mode in case anyone wants to test out this functionality (until this issue is fixed and any necessary concerns are addressed).
Comment #9
gmclelland CreditAttribution: gmclelland commented#8: add-real-preview-of-file-to-format-form10388841038884-1038884-8.patch queued for re-testing.
Comment #11
gmclelland CreditAttribution: gmclelland commentedrelated #1792738: Allow custom file view modes for WYSIWYG display
Comment #12
Devin Carlson CreditAttribution: Devin Carlson commentedAn update of #8.
Comment #13
ParisLiakos CreditAttribution: ParisLiakos commentedI think this is a huge UX win...so i ll go ahead and rtbc this
lets wait a bit before committing though to give time for reactions
Comment #14
John Pitcairn CreditAttribution: John Pitcairn commentedArgh. Sorry, i've been fairly busy and did not test this, but yes I definitely still think this should be optional - most of my users should absolutely not be choosing an image view mode or worrying about the preview size. We frequently alter the displayed size in css anyway on responsive themes, so the preview would not be accurate.
I think this would benefit from a simple Boolean configuration option in field instance settings, off by default.
I have become a little concerned about the way media is constantly adding quite obtrusive features which are turned on by default. UX is considerably dependent on the specific site and its intended users' sophistication and ability - feature proliferation does not equal improved UX. In this case, my users' UX would suffer because you're forcing them to look at something they do not need to consider at all.
Comment #15
aaron CreditAttribution: aaron commented#12: add-real-preview-of-file-to-format-form10388841038884-1038884-12.patch queued for re-testing.
Comment #18
joseph.olstadFixed a while back, thanks @Devin Carlson and @aaron.
Comment #19
joseph.olstad