|Anonymous | Login | Signup for a new account||2019-04-19 05:33 CEST|
|Main | My View | View Issues | Change Log | Roadmap | Docs|
|Viewing Issue Simple Details|
|ID||Category||Severity||Reproducibility||Date Submitted||Last Update|
|0006302||[JEDI VCL] 00 JVCL Components||minor||always||2014-07-09 17:11||2014-12-04 14:57|
|Status||acknowledged||Product Version||Daily / GIT|
|Summary||0006302: JvDBGrid: dgIndicator option currently required to show TitlePopup|
|Description||Assigning a TPopupMenu to the grid's TitlePopup, (which has dgTitle on it's options) but no dgIndicator included, won't trigger the display of such menu when right clicking on the Title.|
Seems an old bug spotted on JvDBGrid.pas 12831 2010-09-05,
however the latest version from
still has it: line 2418
if (Button = mbRight) and
(dgTitles in Options) and (dgIndicator in Options) and
(Cell.Y = 0) then
Why is dgIndicator required to show the TitlePopup?
The third condition of "and (dgIndicator in Options)" seems kinda lost in there.
|Tags||No tags attached.|
|Attached Files||Issue6302.zip [^] (5,612 bytes) 2014-09-03 17:55|
Hi Olivier, could you please check with Andreas if there is any reason to require the presence of dgIndicator to popup the TitlePopup menu?
I suspect that that isn't, and if so, the fix is just to remove the second condition on line 2419, where
" (dgTitles in Options) and (dgIndicator in Options) and"
" (dgTitles in Options) and"
A workaround on every caller side is somewhat messy (catch mouse click, was it on title?, was it a right-click? is it internally reversed? etc...).
|Please provide the zipped sources of a sample application showing this|
|2014-07-09 17:11||cguser||New Issue|
|2014-07-16 13:31||cguser||Note Added: 0021002|
|2014-09-03 11:39||obones||Note Added: 0021034|
|2014-09-03 11:39||obones||Status||new => feedback|
|2014-09-03 17:55||cguser||File Added: Issue6302.zip|
|2014-09-03 17:56||cguser||Note Added: 0021046|
|2014-12-04 14:57||obones||Status||feedback => acknowledged|
|Mantis 1.1.6[^] Copyright © 2000 - 2008 Mantis Group|