Quantcast

qooxdoo 5.0.x Table context menu on header

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

qooxdoo 5.0.x Table context menu on header

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&iu=/4140
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: qooxdoo 5.0.x Table context menu on header

Peter Schneider
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&iu=/4140
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: qooxdoo 5.0.x Table context menu on header

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





------------------------------------------------------------------------------
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&iu=/4140
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: qooxdoo 5.0.x Table context menu on header

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


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&iu=/4140
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: qooxdoo 5.0.x Table context menu on header

Dietrich Streifert
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&iu=/4140
_______________________________________________
qooxdoo-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
Loading...