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

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

Mobile stylesheet broken after update to Fedora 26 #23

Closed fnxweb closed 7 years ago

fnxweb commented 7 years ago

This is almost certainly an issue on my end as opposed to an issue with the skin itself but I wasn't sure where else to raise it. Since I updated my PC to Fedora 26, the mobile skin as been misbehaving.

Roundcubemail itself is OK (and is installed as a Fedora package — I have manually added the skin and jquery_mobile plugin). I've ensure each of the additional components is up to date (git pull), and the mobile plugin is still enabled in my config. I do vaguely recall having to do something else ... odd ... (some sort of extra php-package install via the command line?) when getting this to work first time around (on top of just unpacking the extras) so that may have been undone by the upgrade; unfortunately, I didn't make a note of that and may be misremembering anyway!

Anyway, that's the back-story. Essentially, it seems as if the stylesheets are either not all loading or not properly triggering certain aspects. I'll attach a screenshot, but the core bits are: some missing icons, and the folder view isn't hidden but instead appears as a list at the top of the page, with the usual Inbox entries (missing some icons) below.

I can't see any errors coming out of Apache (2.4.27, php 7.1.7)/roundcube (1.3.0), not can I see anything obvious in the browser network log (nothing not loading). It fails on my mobile, and also desktop Firefox and Chrome (after choosing the mobile view), so it's not browser specific.

This is what I see in my desktop Firefox:

screenshot-www fnxweb co uk 2017-07-30 10-24-18

Any ideas at all on what to look for?

tpayen commented 7 years ago

Hi, the mobile skin current version isn't compatible with roundcube 1.3.0 for the moment.

fnxweb commented 7 years ago

Ah, as I can now see from the sea of such issues on the other repo!

Sorry, I'd convinced myself the version I had was the one OK'd in the README.