search

messagerie-melanie2 / Roundcube-Skin-Melanie2-Larry-Mobile

Mobile Skin for Roundcube
GNU General Public License v3.0
73 stars 18 forks source link

Attachment list is hidden #6

Closed ArnY closed 9 years ago

ArnY commented 9 years ago

When a user is reading a mail with attachments, the attachment list is hidden and so not showing to the user. (.rightcol css object has display attribute set to hidden).

tpayen commented 9 years ago

The rightcol is normally displayed on top of the message body. It has to be displayed when attachment-list is not empty. You are the second one to report this problem but I can't reproduce. What device/web browser do you use ?

ArnY commented 9 years ago

I actually tested with safari on OSX 10.10.3, firefox 41 on linux, Chrome 45 on android. Chromium 45 on linux.

But it's been reported by users using iphones, too.

tpayen commented 9 years ago

Ok actually I reproduce (my fault) I quickly plan a new version to fix that

ArnY commented 9 years ago

Could you provide a quickfix in the meanwhile?

Thanks!

tpayen commented 9 years ago

Here is the patch roundcube_mobile_1510011150.txt

Or you can simply delete this line in roundcube_mobile/js/mobile.js line 257:

 $('#messagecontent .rightcol').hide();
tpayen commented 9 years ago

and don't forget to regenerate the mobile.min.js file

tpayen commented 9 years ago

I've published the new release : https://github.com/messagerie-melanie2/roundcube_mobile/releases/tag/v0.4.3