[SOLVED] TStringgrid popup menu overridden by default popup if cell selected

[Gary Randall]

In my stringgrid I set different PopupMenus in an OnContextPopup event depending on which cell is selected.  This pops up the correct menu when right-clicking on unselected cells, but on selected cells the default menu pops up.  I've tried trapping the right-click in other events with no luck.  I suspect the cell editor is causing this behavior.  Is my suspicion correct?  If so, how do I create and assign a different text editor?  There are a couple of additional features I'd like to have anyway.

Gary

[howardpc]

The default cell editor is based on the underlying OS widgetset editor. So when the mouse is within the widgetset editor in the currently selected cell, the OS widgetset editor swallows the right-click before Lazarus even sees it, and shows the OS editor context menu.
Although you can set alternative cell editors in a Lazarus grid, you will probably find they are all ultimately based on the same OS widget, and so will all exhibit the same behaviour.
Maybe a Lazarus developer (Bart? Jesus?) who maintains these controls will know a way to alter this behaviour without having to write an OS-independent custom-drawn editor that could be an alternative cell editor for you to use.

[engkin]

I suspect the cell editor is causing this behavior.  Is my suspicion correct? 

Yes.

If so, how do I create and assign a different text editor?

Use OnSelectEditor event of the grid to change the pop-up menu of the default Editor, or assign your customized editor to it.

[Gary Randall]

Thanks Howard and engkin.  I studied up on OnSelectEditor and custom editors.  Easier to implement than I thought it would be.  A TMemo has all the editing features I need, including a PopupMenu, - but when I implemented it the default popup overrode the TMemo popup.  So it appears Howard's thoughts are right on.  No matter what event I tested to capture the right-click, they all failed if the cell was previously selected.

Any solution will have to work on cross-platforms.  That will probably make it difficult.  Should I enter this as a bug?

Gary

[engkin]

I don't seem to have the problem on my computer. My grids.pas file is dated May 7, 2013. I suggest you upload a sample app here so other members can confirm the problem, then the same example could be used in a bug report.

[Gary Randall]

Thanks engkin for testing this on your system.  I'm encouraged that it works for you.  I'll make a little test program that we can try.  It may take a few days with the Christmas season on us.

Gary

[Bart]

I opened a bugreport for it.
http://bugs.freepascal.org/view.php?id=27171

Bart

[engkin]

Thanks Bart.

function TCustomGrid.GetDefaultEditor(Column: Integer): TWinControl;
...
    // by default do the editor setup here
    // if user wants to change our setup, this can
    // be done in OnSelectEditor
...

[Bart]

See the disccussion in the bugtracker.

Bart

[engkin]

Thank you, Bart.

Shouldn't any changes be before calling OnSelectEditor, and specifically inside GetDefaultEditor as explained in the comment quoted above?

Preparing the editor before calling OnSelectEditor makes it possible to customize everything, including the pop-up menu used in the editor.

[Bart]

Pleas post comments/suggestions in the bugreport (make my life a little easier  )

Bart

[Gary Randall]

This bug was fixed in Version 1.4.0
Thanks everyone!

[Mr.Paul]

I think this topic is related to my problem, the bug was fixed in 1.4, but I did not understand how to solve my problem at Lazarus 1.8RC4.

I made a new post here: http://forum.lazarus.freepascal.org/index.php/topic,39069.0.html

The default cell editor is based on the underlying OS widgetset editor. So when the mouse is within the widgetset editor in the currently selected cell, the OS widgetset editor swallows the right-click before Lazarus even sees it, and shows the OS editor context menu.
Although you can set alternative cell editors in a Lazarus grid, you will probably find they are all ultimately based on the same OS widget, and so will all exhibit the same behaviour.
Maybe a Lazarus developer (Bart? Jesus?) who maintains these controls will know a way to alter this behaviour without having to write an OS-independent custom-drawn editor that could be an alternative cell editor for you to use.

[jamie]

I just noticed a Context popup that wasn't there in one of my other programs today, after testing The

fix is to implement the OnContextPopUp and set the Handled Property to True.

 In my case it was the Tscrollbar was poping up the Windows 10 Context Menu that wasn't there
before in the past.
 
 The Grid, Edit, ScrollBars ect have this property, you should try it out.

EDIT:
 I did find a work around for your specific issue of the CellEditor not showing the default
context menu if you have a Popup menu assigned while in Editing mode.
 If involves a couple of steps to indentify editor in use during the OnContextPopup event
return as HANDLED and make your choice of popping up the menu..

 Check back if you still have issues.