Right button context menu doesn't work in Opera or Chrome

[rossvernal]

When using the context menu available from the right button, we have noticed that this doesn't work for Opera 9.64 and 10.10 and Chrome 2.0, 3.5 and 4.0. It works for the same Weblayout in IE7, IE8, FF3.5 and Safari 4.

These are only the browsers we have tested this in.

See screenshots.

[rossvernal]

The failed right click in Opera

[rossvernal]

The right click menu, working as expected in Safari.

[rossvernal]

Opera with MapGuide 2.2 Beta 1 - RH Menu not working

[rossvernal]

Google Chrome with MapGuide 2.2 Beta 1 - working context menu

[rossvernal]

Replying to rossvernal:

When using the context menu available from the right button, we have noticed that this doesn't work for Opera 9.64 and 10.10 and Chrome 2.0, 3.5 and 4.0. It works for the same Weblayout in IE7, IE8, FF3.5 and Safari 4.

These are only the browsers we have tested this in.

See screenshots.

Having installed MapGuide 2.2 Beta 1 - I have tested the context menu in the latest releases of Chrome and Opera. The context menu looks good and works fine in Chrome, but it doesn't in Opera.

[tomfukushima]

The reason it is working in Chrome now is because MapGuide Enterprise supports Chrome and so work was done by Autodesk to address this issue. One possible way to get this resolved in Opera is to donate to the project and try to direct the funding to resolve this ticket.

[zspitzer]

I'll take this, got a partial fix working with the menu's showing

The ZoomIn and ZoomOut functions don't seem to work either in Opera

[zspitzer]

first cut of patch (untested, no local build available)

[zspitzer]

v2 patch

[zspitzer]

Patch now works in Opera, however the zoom in and zoom out don't work now

[zspitzer]

Ok the zoomin and zoomout do work, i was thinking that selecting thos options would immediately zoomin or zoomout, rather than just changing the mouse action.

Alas, Opera in this day and age still doesn't support custom cursors

​http://stackoverflow.com/questions/447038/opera-and-custom-cursor-in-css

[tomfukushima]

Submitted to trunk: r5208 2.2: r5029 Please note that the patch has a syntax error - code in browserdetect.js was missing a brace. Also spacing was off. (Zac please see if you can find a way to prevent these syntax errors from slipping in as they are unexpected and take time to resolve.)

[tomfukushima]

BTW, nice technique to use obj[attr] in place of the this.attr.