[DISCUSSION] BeforeDrawRow event

Hello,

Hi Aaron,

I've had a chance to try a couple of tests so far with 14.0.0 beta. Some notes:

1. BeforeDrawRow only fires on MouseMove for the cell that the mouse has entered. For hot-tracking purposes, maybe it should fire for the cell that the mouse has left as well?
2. Even though BeforeDrawRow fires on MouseMove, changes made to Item.Caption or Metrics.Text do not get updated on screen.
3. On MouseDown/MouseUp, BeforeDrawRow fires multiple times (2-3 times??)  for every visible cell on the report. I can't imagine why this should happen, but there may be a good reason? I think it would be better to fire once for each recorditem in the previously selected rows that aren't currently selected, and then fire once for every recorditem in the newly selected rows that used to be unselected. This would cover all of the scenarios that I have imagined so far, but further exploration is needed here.
4. Even though the BeforeDrawRow fires on MouseDown/Up, changing the Item.Caption doesn't seem to get displayed immediately (the screen update is always one step behind).
   

The fact that updates to the RC after changing the Item.Caption in BeforeDrawRow always seems to be behind makes me think that I am missing something (Redraw method doesn't work since it cause BeforeDrawRow to fire continuously, UpdateRecord killed the IDE for me - so what is the right way to get an immediate ReportItem update on-screen after changing the Item.Caption in the BeforeDrawRow event?).

Maybe what we need is a BeforeDrawRowFlags property with flags to control when BeforeDrawRow is fired based on our needs? Something like:

xtpBeforeDrawRowMouseEnter - Fire when mouse enters a RecordItem. Only fire once for that recorditem.
xtpBeforeDrawRowMouseLeave - Fire when mouse leaves a RecordItem. Only fire once fo that recorditem.
xtpBeforeDrawRowMouseMove - Fire when mouse is moved within a RecordItem. Only fire for that recorditem.
xtpBeforeDrawRowMouseDown - Fire when mouse button is depressed while over a RecordItem. only fire once for that recorditem
xtpBeforeDrawRowMouseUp - Fire when mouse is released while over a RecordItem. Only fire once for that recorditem.
xtpBeforeDrawRowSelected - Fire when a RecordItem parent Record becomes selected. Only fire once for that recorditem.
xtpBeforeDrawRowDeselected - Fire when a RecordItem parent Record becomes deselected. Only fire once for that recorditem.

Flags could of course be combined to fire under multiple circumstances. It would be nice if the BeforeDrawRow event would also pass the flag that caused the firing as a parameter (of course, for Binary compatibility, this would mean we would need a different event, perhaps BeforeDrawRowEx).

These flags would cover all of the cases I have thought of so far where you would want BeforeDrawRow to fire, but there may be other times - any ideas on extensions for this list?

Here's the test project I was using:

uploads/20100130_145004_RcBeforeDrawRow.zip