search

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

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

Reply-all (Reply list) doesn't fill in all recipients #12

Closed florianbeer closed 8 years ago

florianbeer commented 8 years ago

Hello, first let me say thank you for this awesome skin (and plugin) for Roundcube. It is really helpful to have a good looking and functional mobile version for my webmail.

One of my email users recently brought to my attention that when selecting "Reply list" on an email with multiple recipients, it only fills in the first email address into the "To" filed in the new email. All the others would have to be copied over manually. I couldn't track down why this is in the source code, so I thought I'd ask here for assistance.

Thanks!

tpayen commented 8 years ago

Hello, do you have this problem only with the mobile skin ? I'm not sure if it's related to mobile skin

florianbeer commented 8 years ago

Yes, only with the mobile skin. I just saw that the standard Roundcube skin distinguishes between "Reply list" and "Reply all", on mobile skin I only see "Reply list".

That might be a clue?

If you want, I can make a temporary account for you and you can log in and check it out yourself.

tpayen commented 8 years ago

What's the value for the option in Preferences > Composing Messages > Default action of [Reply all] button ?

florianbeer commented 8 years ago

I have set the following in my config/config.inc.php

$config['reply_mode'] = 1;
$config['reply_all_mode'] = 1;

and both of them are disabled from showing in the interface (via $config['dont_override'] array). Maybe that's why it's not working as expected?

...going to test this out now.

florianbeer commented 8 years ago

Yes, that's it!

I've removed all "reply" settings in config.inc.php and now "Reply all" is showing and working as advertised in mobile view.

Thanks for your help 👍