Roundcube Community Forum

 

Melanie2 Larry Mobile skin v0.4.7 / Roundcube 1.2.3

Started by tpayen, May 20, 2015, 12:45:32 PM

Previous topic - Next topic

twisterbr

I downloaded this skin and its just awesome! You guys did a great job!
Now i'm trying to edit the skin for personal styiling and want to ask if anyone can help me with someth.

Is it possible to edit the roundcube object textarea?

I want to make it look like the image attached, by putting a placeholder on the compose fields.
Is it possible to do it on compose.html of the skin?
<roundcube:object name="composeHeaders" part="to" form="form" id="_to" rows="1" tabindex="2" />

thx

tpayen

Hi,
It seems that the Roundcube framework doesn't support placeholder for textarea.

In program/lib/Roundcube/html.php
'placeholder' isn't in the "$allowed" array for html_textarea class, but is present for html_inputfield class.
You can try to add it and suggest the correction to the Roundcube team



twisterbr

I tried putting placeholder on the allowed array and then on the object on compose.html, but it didn't work.
I'll keep trying.
They should include it on the framework anyway since it's a basic parameter.

Thanks for the tip!

SKaero


jokela

#34
Greetings. This skin looks amazing but it seems that I'm getting some troubles to get it working.

I'm using roundcube 1.1.2 running on Apache2 with  PHP 5.3.6

I followed the instructions but when I use the plugin, it throws PHP errors.

I used this line in config.inc.php to activate the plugin

$config['plugins'] = array('mobile');

This are the errors found in Apache's error.log:

PHP Parse error:  syntax error, unexpected '[', expecting ')' in /var/www/webmail/plugins/mobile/mobile.php on line 187
PHP Stack trace:
PHP   1. {main}() /var/www/webmail/index.php:0
PHP   2. rcmail::get_instance() /var/www/webmail/index.php:43
PHP   1. {main}() /var/www/webmail/index.php:0
PHP   2. rcmail::get_instance() /var/www/webmail/index.php:43
PHP   3. rcmail->startup() /var/www/webmail/program/include/rcmail.php:77
PHP   4. rcube_plugin_api->load_plugins() /var/www/webmail/program/include/rcmail.php:130
PHP   4. rcube_plugin_api->load_plugins() /var/www/webmail/program/include/rcmail.php:130
PHP   5. rcube_plugin_api->load_plugin() /var/www/webmail/program/lib/Roundcube/rcube_plugin_api.php:108
PHP   5. rcube_plugin_api->load_plugin() /var/www/webmail/program/lib/Roundcube/rcube_plugin_api.php:108

Any suggestion about what should I try?

Thanks in advance and go on with the nice job.


PD: A very similar error is thrown when using the NO mobile version of the skin (roudcube_skin_melanie2_larry) Maybe related?
This is thrown in this case:

PHP Parse error:  syntax error, unexpected '[' in /var/www/webmail/plugins/melanie2_larry/melanie2_larry.php on line 46
PHP Stack trace:
PHP   1. {main}() /var/www/webmail/index.php:0
PHP   2. rcmail::get_instance() /var/www/webmail/index.php:43
PHP   3. rcmail->startup() /var/www/webmail/program/include/rcmail.php:77
PHP   2. rcmail::get_instance() /var/www/webmail/index.php:43
PHP   3. rcmail->startup() /var/www/webmail/program/include/rcmail.php:77
PHP   4. rcube_plugin_api->load_plugins() /var/www/webmail/program/include/rcmail.php:130
PHP   5. rcube_plugin_api->load_plugin() /var/www/webmail/program/lib/Roundcube/rcube_plugin_api.php:108


PD2: I was doing some extra research and it could be related with changes in syntax for dereferencing function? (PHP 5.3 vs PHP 5.4)
If this is the reason, it would be nice to have some kind of warning about PHP version requirements of the skin :)

More info about dereferencing changes: http://php.net/manual/en/language.types.array.php

Please, let me know if you intend to do it compatible with 5.3 (using the old style with temporary variable). In my case moving to higher PHP could be problematic for other php based pages hosted in the same server.

jvenezia

Hi-
Was able to make skin work once but could not upload attachments.  uninstalled everything and reinstalled.  Now attachments work, bu the mobile plugin just shows a blank screen when enabled.  Have you seen this before?  Any suggestions?  Centos 6.7, apache 2.2, PhP 5.3.3.  If I select the melanie2_larry_mobile skin manually, no css seems to load.  Looks like the templates without CSS.

SKaero

The skin plugin is using PHP 5.4.x syntax, you'll have to update PHP to use the plugin.

jokela

Finally I migrated the webmail service to a different server, and I can confirm that this is compatible with PHP 5.4, but NO with PHP 5.3.

As I suggested in the other post, please, consider to include a big warning of php version requirements so anyone else know and not spend time with old PHP versions.


Nice work and great usability, thanks a lot for your efforts.

tpayen

Hi,
Indeed mobile skin currently needs PHP 5.4, but this is just a syntax problem. Next time I update the project, I'll correct it.
And thanks for your support :-)
tpayen

mrsuchy

#39
Where I can find my attachments in email view ?

---edit
I see attachments when I display a email and refresh browser page.

tpayen

Attachments are listed above the message body
You must use the lastest version (0.4.4) cause versions < 0.4.3 are bugged with attachments

skin v0.4.4 : https://github.com/messagerie-melanie2/roundcube_skin_melanie2_larry_mobile/releases/tag/v0.4.4
plugin v0.4.4 : https://github.com/messagerie-melanie2/roundcube_mobile/releases/tag/v0.4.4


tiagoalam

Thank you for theme, really good.

Please, I have a problem when accessed through Safari on iOS, the email list is not displayed, it is necessary to recharge to be shown.
You could look at it? thank you!

tpayen

There is a incompatibility between Safari, Jquery Mobile and Roundcube. Unfortunately I can't correct it directly in the mobile plugin.
But you can find all information to solve the problem in this github issue : https://github.com/messagerie-melanie2/roundcube_skin_melanie2_larry_mobile/issues/4