Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#180 closed defect (fixed)

Contextmenu cant be attached to specific nodes and it doesnt work in Konqueror

Reported by: fredrik.j at bredband.net Owned by: anonymous
Priority: high Milestone:
Component: Widgets Version:
Keywords: Cc:
Blocked By: Blocking:

Description

Contextmenu have no way to set menu to specific nodes instead of the hole page. And it does not work in konqueror3.4.3 which doesnt support oncontextmenu event.

I have fixed and implemented this in the supplied pathes. It basically use onmousedown in konqueror, and oncontextmenu in moz, IE etc. And attach event listeners to nodeIds in targetNodesId array, if this is empty it defaults to body just like before.

Hope you can use them!

They are changed files from rev1850.

Regards Fredrik Johansson

Attachments (4)

ContextMenu.js (3.1 KB) - added by fredrik.j at bredband.net 14 years ago.
patched file ContextMenu? with new features and bugfixes
test_ContextMenu.html (1.5 KB) - added by fredrik.j at breband.net 14 years ago.
show the new features and fix bug 178
ContextMenu.2.js (4.4 KB) - added by fredrik dot j at bredband dot net 14 years ago.
this file file works better than my previous fix (I hope)
HtmlContextMenuTemplate.html (69 bytes) - added by fredrik dot j at bredband dot net 14 years ago.
Needed to remove onclick event here to get konqueror to work

Download all attachments as: .zip

Change History (7)

Changed 14 years ago by fredrik.j at bredband.net

Attachment: ContextMenu.js added

patched file ContextMenu? with new features and bugfixes

Changed 14 years ago by fredrik.j at breband.net

Attachment: test_ContextMenu.html added

show the new features and fix bug 178

Changed 14 years ago by fredrik dot j at bredband dot net

Attachment: ContextMenu.2.js added

this file file works better than my previous fix (I hope)

Changed 14 years ago by fredrik dot j at bredband dot net

Needed to remove onclick event here to get konqueror to work

comment:1 Changed 14 years ago by fredrik dot j at bredband dot net

Sorry for all the trouble but file nr ContextMenu?.2.html works better with konqueror and it fixes bug 179.

I hope you can use the fixes, I tried not to delete any of your code only comment out. I tested them in: FF 1.0.7 IE 6 under wine konqueror3.4.3

/ Fredrik Johansson

comment:2 Changed 14 years ago by alex

Resolution: fixed
Status: newclosed

this is fixed with [2048] thanks to Fredrik and Cal.

Thanks guys, it looks great.

comment:3 Changed 13 years ago by (none)

Milestone: 0.2release

Milestone 0.2release deleted

Note: See TracTickets for help on using tickets.