Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#1274 closed defect (fixed)

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

Reported by: rossvernal Owned by: zspitzer
Priority: high Milestone:
Component: AJAX Viewer Version: 2.2.0
Severity: major Keywords:
Cc: jng External ID:

Description

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.

Attachments (6)

opera.gif (96.7 KB) - added by rossvernal 9 years ago.
The failed right click in Opera
safari-working.gif (60.6 KB) - added by rossvernal 9 years ago.
The right click menu, working as expected in Safari.
opera2.2beta1.png (45.3 KB) - added by rossvernal 8 years ago.
Opera with MapGuide 2.2 Beta 1 - RH Menu not working
chrome2.2beta1.png (1.4 MB) - added by rossvernal 8 years ago.
Google Chrome with MapGuide 2.2 Beta 1 - working context menu
refactored getAttributeNodeValue.patch (4.7 KB) - added by zspitzer 8 years ago.
first cut of patch (untested, no local build available)
refactored getAttributeNodeValue.2.patch (4.7 KB) - added by zspitzer 8 years ago.
v2 patch

Download all attachments as: .zip

Change History (14)

Changed 9 years ago by rossvernal

Attachment: opera.gif added

The failed right click in Opera

Changed 9 years ago by rossvernal

Attachment: safari-working.gif added

The right click menu, working as expected in Safari.

Changed 8 years ago by rossvernal

Attachment: opera2.2beta1.png added

Opera with MapGuide 2.2 Beta 1 - RH Menu not working

Changed 8 years ago by rossvernal

Attachment: chrome2.2beta1.png added

Google Chrome with MapGuide 2.2 Beta 1 - working context menu

comment:1 in reply to:  description Changed 8 years ago by rossvernal

Version: 2.1.02.2.0

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.

comment:2 Changed 8 years ago by 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.

comment:3 Changed 8 years ago by zspitzer

Owner: set to 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

Changed 8 years ago by zspitzer

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

comment:4 Changed 8 years ago by jng

Cc: jng added

Changed 8 years ago by zspitzer

v2 patch

comment:5 Changed 8 years ago by zspitzer

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

comment:6 Changed 8 years ago by 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

comment:7 Changed 8 years ago by tomfukushima

Resolution: fixed
Status: newclosed

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.)

comment:8 Changed 8 years ago by tomfukushima

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

Note: See TracTickets for help on using tickets.