|Summary:||analog clock: no context menu|
|Product:||plasma4||Reporter:||Maciej Pilichowski <bluedzins>|
|Component:||general||Assignee:||Plasma Bugs List <plasma-bugs>|
|Latest Commit:||Version Fixed In:|
Description Maciej Pilichowski 2007-12-23 20:35:06 UTC
Version: (using KDE KDE 3.97.0) There is only applet handle available. There is only applet handle available.
Comment 1 Jason Stubbs 2007-12-24 04:44:27 UTC
SVN commit 752290 by jstubbs: Derive from Applet rather than Containment so that all the Applet goodies such as context menus are available BUG: 154552 M +1 -1 clock.cpp M +2 -2 clock.h WebSVN link: http://websvn.kde.org/?view=rev&revision=752290
Comment 2 Aaron J. Seigo 2007-12-27 15:43:59 UTC
this patch is incorrect. deriving from containment should not interfere with context menus. investigating why this would be the case is really what needs to happen. i'd guess that either there is a missing isContaintment() somewhere or else maybe the qgraphicsitem_cast in Containment::contextMenuEvent isn't working as expected.
Comment 3 Jason Stubbs 2007-12-27 16:15:17 UTC
SVN commit 753472 by jstubbs: Don't try and display the containment context menu when isContainment() is false. Also, call Applet's contextMenuEvent() rather than calling QGraphicsItem's directly. CCBUG: 154552 M +2 -2 containment.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=753472
Comment 4 Jason Stubbs 2007-12-27 16:16:04 UTC
SVN commit 753474 by jstubbs: Change back to a containment now that the context menu displays correctly CCBUG: 154552 M +1 -1 clock.cpp M +2 -2 clock.h WebSVN link: http://websvn.kde.org/?view=rev&revision=753474
Comment 5 Jason Stubbs 2007-12-27 16:20:25 UTC
There's a bug in qt4.3 that won't be fixed until qt4.4 that context menu events aren't propogated to parent items. I previously moved the workaround that was in Applet into Widget, but Containment hadn't been updated for it.