Re: [qooxdoo-devel] qooxdoo 5.0.x Table context menu on header
Hi Peter, investigating the commit history reveals the following commit: https://github.com/qooxdoo/qooxdoo/commit/36ae2c5b94f3a1ea6207b909465f66bb9b3dce70 which says: [Bug #8757] Special handling for layout poperty top = 0 removed. Cont… …ext menu handling on table widget overriden, will be handeled by scroll pane Here is the corresponding bugzilla entry (yes opened by me): http://bugzilla.qooxdoo.org/show_bug.cgi?id=8757 The idea was to have a contextmenu which only shows up if your are right clicking over cells. Additionally you have to set the pane scroller property contextMenuFromDataCellsOnly to false: http://demo.qooxdoo.org/current/apiviewer/#qx.ui.table.pane.Scroller~contextMenuFromDataCellsOnly!property Here is a working playground example: http://tinyurl.com/jb7e2ck Regards Dietrich -- Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140 ___ qooxdoo-devel mailing list qooxdoo-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Re: [qooxdoo-devel] qooxdoo 5.0.x Table context menu on header
I'm not sure why exactly qx.ui.table.Table overrides _onContextMenuOpen with
an empty implementation, but until I know here's what I have to do:
http://tinyurl.com/gm2az7a (qooxdoo 5.0.x work-around)
I "re-enabled" the super-super-class' method in a derived Table class like:
// ---
qx.Class.define("my.Table", {
extend: qx.ui.table.Table,
members : {
// overridden
_onContextMenuOpen : function (e) {
//
// qx.ui.table.Table overrides an empty implementation, but we need this
// to show the context menu on header cells.
//
// Therefore we call the "super.super" method (qx.ui.core.Widget)
//
qx.ui.core.Widget.prototype._onContextMenuOpen.call(this, e);
}
}
});
// ---
Regards,
Peter
Am 2/8/2016 um 5:15 PM schrieb Peter Schneider:
> Hi there,
>
> after an upgrade to qooxdoo 5.0.1 adding a context menu to a table does not
> render it when (right-)clicked on header cells / pane.
>
> I can see that the (empty) override of _onContextMenuOpen at qx.ui.table.Table
> prevents this from happening.
>
> What do I need to do to make the context menu (re-)appear on table header
> cells?
>
> Or is this a regression that I can't work around without patching the
> framework...
>
> For comparison:
> 4.0.x : http://tinyurl.com/z8ch32p ( context menu on header)
> current : http://tinyurl.com/jop2fck (*no* context menu on header)
>
> Regards,
> /Peter
>
--
** Unsere Veranstaltungen 2016 - Termine vormerken!
Logimat, Stuttgart in Halle 7, Stand A05 8.-10.3.2016
T-Matik, Greven 10.05.2016
InnoMATIK 2016, 4. Innovationsforum Telematik 18.-19.05.2016
>> Um Anmeldung wird gebeten: http://tis-innomatik.de/
BWVL-Jahrestagung im Oktober 2016
** Kennen Sie schon unseren Blog?
telematics-magazine.com
** Interesse an unseren Unternehmensneuigkeiten? >> Newsletteranmeldung:
http://www.tis-gmbh.de/de/newsletter/anmeldung
--
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140
___
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Re: [qooxdoo-devel] qooxdoo 5.0.x Table context menu on header
Hi Dietrich, thanks for the explanations. So the only way to get the context menu back at table *header* is by overriding the empty override, like I proposed in my last post, right? /Peter Am 2/9/2016 um 11:52 AM schrieb Dietrich Streifert: > Hi Peter, > > investigating the commit history reveals the following commit: > > https://github.com/qooxdoo/qooxdoo/commit/36ae2c5b94f3a1ea6207b909465f66bb9b3dce70 > > which says: > > [Bug #8757] Special handling for layout poperty top = 0 removed. Cont… > …ext menu handling on table widget overriden, will be handeled by > scroll pane > > Here is the corresponding bugzilla entry (yes opened by me): > > http://bugzilla.qooxdoo.org/show_bug.cgi?id=8757 > > The idea was to have a contextmenu which only shows up if your are right > clicking over cells. > > Additionally you have to set the pane scroller property > contextMenuFromDataCellsOnly to false: > > http://demo.qooxdoo.org/current/apiviewer/#qx.ui.table.pane.Scroller~contextMenuFromDataCellsOnly!property > > Here is a working playground example: > > http://tinyurl.com/jb7e2ck > > Regards > Dietrich >[...] -- ** Unsere Veranstaltungen 2016 - Termine vormerken! Logimat, Stuttgart in Halle 7, Stand A05 8.-10.3.2016 T-Matik, Greven 10.05.2016 InnoMATIK 2016, 4. Innovationsforum Telematik 18.-19.05.2016 >> Um Anmeldung wird gebeten: http://tis-innomatik.de/ BWVL-Jahrestagung im Oktober 2016 ** Kennen Sie schon unseren Blog? telematics-magazine.com ** Interesse an unseren Unternehmensneuigkeiten? >> Newsletteranmeldung: http://www.tis-gmbh.de/de/newsletter/anmeldung -- Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140 ___ qooxdoo-devel mailing list qooxdoo-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Re: [qooxdoo-devel] qooxdoo 5.0.x Table context menu on header
No, you can also add a contextmenu to the table header: http://tinyurl.com/jsmj63k Am 09.02.2016 um 13:38 schrieb Peter Schneider: > Hi Dietrich, > > thanks for the explanations. > > So the only way to get the context menu back at table *header* is by > overriding the empty override, like I proposed in my last post, right? > > /Peter > > -- Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151=/4140 ___ qooxdoo-devel mailing list qooxdoo-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
