Am 17.11.2012 18:13, schrieb Thomas Bruederli:
Michael Heydekamp wrote:
Just displaying the text "Thumbnail" (well, how would you translate this into German ;) ?) somewhere above/below the picture...? In contradiction to A.L.E.C's post, the size is OK to me, as in most cases this relatively large thumbnail size would often not even lead me to display/load the picture in full size at all. This is pretty convenient, IMHO.
First of all, the size of the thumbnails is configurable through the 'image_thumbnail_size' option.
Ah, this I didn't know.
- Is it planned to provide a second link (next to the "Download" link),
which simply displays the image - full size or fit-to-screen, and probably in a new window - within Roundcube? We know from the past that Roundcube can do that. ;-) That would at least be much more convenient rather than downloading the image and then loading it into the local app being associated to this file type on this particular system - if such an app is available at all (think of Internet Cafés and such).
You can click on the thumbnail itself to open up the full size image. Mabye an additional link "Show" next to "Download" would fix this.
And I just added these explicit links to "Show" an image attachments right next to the "Download" links.
Great, thanks.
At least it currently doesn't work the way you're describing in IE8/Win7. I can click on the thumbnail, but instead of opening up the full size image within Roundcube or the browser, I'm getting a dialogue "Möchten Sie diese Datei öffnen oder speichern?" ("Do you want to open or save this file?"). If I click on "Öffnen" (= Open), then the image is loaded into the "Windows-Fotoanzeige" - and this is exactly what I would like to avoid, as this is an app on the local machine.
[...]
But I had a closer look at it with IE9 on Win7 and indeed found out a totally weird behavior there, which caused all attachments to be downloaded directly: the list of supported mimetypes, which is passed to the browser as array mimetypes:["text/plain",...] is, for some reason only Microsoft knows, treated as an object here. So .indexOf() didn't work as expected.
That's really a bummer! I didn't find out why IE randomly treats array syntax as Object instead of Array. I workarounded it in https://github.com/roundcube/roundcubemail/commit/54cc75f28d75d01eb10c0be51e...
I just saw it in the svn-list. This again proves how important testing is.
I will test it this night or tomorrow. We are automatically pulling the latest git version between 1h and 2h.
Again, great that you found (and fixed) this weird behaviour of IE.
Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany