#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)
Change History (7)
Changed 15 years ago by
Attachment: | ContextMenu.js added |
---|
Changed 15 years ago by
Attachment: | test_ContextMenu.html added |
---|
show the new features and fix bug 178
Changed 15 years ago by
Attachment: | ContextMenu.2.js added |
---|
this file file works better than my previous fix (I hope)
Changed 15 years ago by
Attachment: | HtmlContextMenuTemplate.html added |
---|
Needed to remove onclick event here to get konqueror to work
comment:1 Changed 15 years ago by
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 15 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
this is fixed with [2048] thanks to Fredrik and Cal.
Thanks guys, it looks great.
patched file ContextMenu? with new features and bugfixes