Roundcube Community Forum

 

Right Click Functionality

Started by mfallon, February 28, 2009, 02:18:13 PM

Previous topic - Next topic

Do you think right click funtionality is a good idea?

es
181 (91.9%)
o
6 (3%)
aybe
10 (5.1%)
on\'t Care / Know
0 (0%)

Total Members Voted: 193

tsjolshagen

Quote from: dziobak;29538tsjolshagen: try it with IE.
I have two days old version and everythings ok in ff and ie.

Think you may have missed a salient point regarding using IE on these systems; they're running Linux (no IE)

tsjolshagen

Quote from: ABerglund;29539No, it's working fine for me. But this sounds suspiciously like a problem I had some time back with a conflict between contextmenu and compose_addressbook, which I see is enabled in the list above. Try disabling compose_addressbook and see if that helps. If so, you may want to verify that compose_addressbook is up to date.

Before I saw this I installed firebug and started disabling plugins in an attempt to locate possible collisions. The plugin that is "colliding" (enabled alongside contextmenu means the UI stalls) is compose_newwindow.

With Firebug installed and compose_newwindow + contextmenu enabled I get one error message:

"uncaught exception: Syntax error, unrecognized expression: Syntax error, unrecognized expression: a"

Hth,

// Thomas

tsjolshagen

When I change the folder and attempt to delete multiple messages from the new folder using right-click, i get the following javascript error:

e is null
http://server/directory/program/js/common.js?s=1281197014
Line 339

tsjolshagen

Quote from: tsjolshagen;29558When I change the folder and attempt to delete multiple messages from the new folder using right-click, i get the following javascript error:

e is null
http://server/directory/program/js/common.js?s=1281197014
Line 339

Also applies when attempting to delete a single message in a new folder

JohnDoh

Quote from: tsjolshagen;29557With Firebug installed and compose_newwindow + contextmenu enabled I get one error message:

"uncaught exception: Syntax error, unrecognized expression: Syntax error, unrecognized expression: a"

The bug is in the compose_newwindow please contact the author of that plugin.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

JohnDoh

Quote from: tsjolshagen;29558When I change the folder and attempt to delete multiple messages from the new folder using right-click, i get the following javascript error:

e is null
http://server/directory/program/js/common.js?s=1281197014
Line 339

This has been fixed in the repo.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

mattfox27

#141
Hello i just installed this in RC .4 and it doesn't work.  I tried Firefox and IE i get no right click menu.  Does this work for .4?

I'm new to RC this is how its supposed to look in conf file
$rcmail_config['plugins'] = array(contextmenu);

Or should it have single quotes ('contextmenu')

I figured it out...im running a plugin called RCguard which opens a Captcha windows after X amount of failed plugins and that kills all my plugins.  Anyone run into this issue?

JohnDoh

Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

Cooldown

#143
To all who uses the contextmenu plugin with composenewwindow and got the following error with firebug "uncaught exception: Syntax error, unrecognized expression: Syntax error, unrecognized expression: a" and spend too much time with searching in this forum.

(@offtopic: i hate the captcha and the "answer the question" stuff! This will not prevent the forum from spammers cause the captcha is weak and the questions rotating ... not hard to test against a string. The only thing it does is to prevent users from using the search function with ease)

JohnDoh has given the answer in the following Post http://www.roundcubeforum.net/7-third-party-contributions/46-api-based-plugins/7169-compose-new-window-plugin.html#post29572

To keep Things at one place

Quote from: JohnDoh;29572[..] The error comes from the JS in the plugin itself, in closewindow.js, this file contains the code which interacts with the context menu. `li.reply :a` and the 3 other similar lines are wrong, take the `:` away and everything works like normal. AFAIK you can't use selectors like that.

(Jepp there is no :a selector xD)

Btw .. my quick and dirty fix was to replace rcmail.contextmenu_command_handlers = new Object(); with rcmail.contextmenu_command_handlers = ''; in contextmenu.js ^^ works too but as i walked over the source code in contextmenu.js i think this will break the external commands processing.

Just for the record the quick and dirty way seems to work in most cases but i and you should prefer JohnDoh's fix cause this is the source where the error comes from. The Q&D way can cause other problems in other plugins that uses similar functionality.

Regards, Cooldown

ABerglund

In my test environment, I'm using the latest version of the plugin (20 Aug 2011) in 0.6-beta. I'm seeing an issue I've seen for quite some time with spinning progress wheel during a "mark message" operation. Can anyone else duplicate this? Here are the steps:

Go to any folder with multiple unread messages. Close the Preview Pane if you use it.

Open one message to view it.

While the message is open, right-click the folder that contains the open message, and select "mark all as read".

In my experience, the progress wheel will drop and spin ("markingmessage"), but it never refreshes the folder list, and the dialog never goes away. You need to click into a different folder to escape the hang, at which point all the messages will have been marked as you requested.
Arne Berglund
SysAdmin, Internet Services
Lane Education Service District
Eugene, OR, USA

JohnDoh

thanks Arne, that bug has been fixed.
Roundcube Plugins: Contextmenu, SpamAssassin Prefs, and more...

ABerglund

I can confirm. While all the complete package download links seem to pull the same version I was testing, I could copy and paste the two changed files (contextmenu.js and contextmenu.php) from github into my test instance, and the problem is corrected. Thanks!
Arne Berglund
SysAdmin, Internet Services
Lane Education Service District
Eugene, OR, USA

rosali

Yes, it looks like the zip has not been updated. Only the repo version corrects the issue.
Regards,
Rosali

anglebaby

Also applies when attempting to delete a single message in a new folder

Schmatze

I'm using roundcube 0.5.4 and if I have contextmenu enabled I can't open the addressbook.

QuotePHP Fatal error:  Call to undefined function html_identifier() in /.../plugins/contextmenu/contextmenu.php on line 272