Re[2]: Move notifies around to match native

2005-04-15 Thread Vitaliy Margolen
Thursday, April 14, 2005, 2:49:45 PM, you wrote: > {Set,Get}DCBrushColor do something different to what you expect. They > set the DC_BRUSH colour, not the colour of the current brush selected > into the DC. You can get select it into the DC using > SelectObject(GetStockObject(DC_BRUSH)). The con

Re: Move notifies around to match native

2005-04-14 Thread Robert Shearman
Vitaliy Margolen wrote: Thursday, April 14, 2005, 11:50:50 AM, you wrote: Vitaliy Margolen wrote: Ok, I need help here. I spend half a day trying to fix this. I moved send notify after color and font setup in TREEVIEW_DrawItem. It's getting closer to native. But! No matter what I try, I can

Re[2]: Move notifies around to match native

2005-04-14 Thread Vitaliy Margolen
Thursday, April 14, 2005, 11:50:50 AM, you wrote: > Vitaliy Margolen wrote: >>Tuesday, April 12, 2005, 10:57:02 PM, you wrote: >> >> >>>On Tue, Apr 12, 2005 at 04:13:32PM -0600, Vitaliy Margolen wrote: >>> >>> Well, yes and no. This is custom draw callback/notify. It shouldn't change

Re: Move notifies around to match native

2005-04-14 Thread Robert Shearman
Vitaliy Margolen wrote: Tuesday, April 12, 2005, 10:57:02 PM, you wrote: On Tue, Apr 12, 2005 at 04:13:32PM -0600, Vitaliy Margolen wrote: Well, yes and no. This is custom draw callback/notify. It shouldn't change control's parameters, such as color. At least on a permanent basis. This is w

Re[2]: Move notifies around to match native

2005-04-14 Thread Vitaliy Margolen
Tuesday, April 12, 2005, 10:57:02 PM, you wrote: > On Tue, Apr 12, 2005 at 04:13:32PM -0600, Vitaliy Margolen wrote: >> Well, yes and no. This is custom draw callback/notify. It shouldn't change >> control's parameters, such as color. At least on a permanent basis. This is >> what >> set color mes

Re: Move notifies around to match native

2005-04-12 Thread Dimitrie O. Paun
On Tue, Apr 12, 2005 at 04:13:32PM -0600, Vitaliy Margolen wrote: > Well, yes and no. This is custom draw callback/notify. It shouldn't change > control's parameters, such as color. At least on a permanent basis. This is > what > set color messages for. Agreed. But judging by what listview is doi

Re[2]: Move notifies around to match native

2005-04-12 Thread Vitaliy Margolen
Tuesday, April 12, 2005, 3:09:57 PM, you wrote: > On Tue, Apr 12, 2005 at 12:24:11PM -0600, Vitaliy Margolen wrote: >> Also small fix for custom draw color. Native does not use colors returned >> from >> NM_CUSTOMDRAW notify. > This is odd, and it deserves at the very least a comment in the > cod

Re: Move notifies around to match native

2005-04-12 Thread Dimitrie O. Paun
On Tue, Apr 12, 2005 at 12:24:11PM -0600, Vitaliy Margolen wrote: > Also small fix for custom draw color. Native does not use colors returned from > NM_CUSTOMDRAW notify. This is odd, and it deserves at the very least a comment in the code, but more preferably a unit test. -- Dimi.