Re: Corrected the determination of capturing inside EDIT_WM_MouseMove function.

2005-05-24 Thread taro-x
2005-05-24 Kouji Sasaki <[EMAIL PROTECTED]> > > Hence, the modification was made so that the text selection by mouse > movement will only occur when edit control itself sets the capture. > > OK, I'm sold, but can you please resubmit the patch with > an appropriate comment expl

Re: Corrected the determination of capturing inside EDIT_WM_MouseMove function.

2005-05-20 Thread taro-x
2005-05-19 Kouji Sasaki <[EMAIL PROTECTED]> > > With only GetCapture() != es->hwndSelf statemet, the string selection will > occur > > even if the capturing is done by external processes. > > So what you are saying is that an external process captures the mouse, > and someh

Re: Corrected the priority of operators for LISTVIEW_GetNextItem function.

2005-05-19 Thread taro-x
2005-05-19 Kouji Sasaki <[EMAIL PROTECTED]> > > Changelog: > > > > * dlls/comctl32/listview.c > > > > Corrected the priority of operators for LISTVIEW_GetNextItem function. > > > --- dlls/comctl32/listview.c 2005-04-14 22:58:24.0 +0900 > > +++ dlls

Re: Added the feature to change row height in owner draw controls.

2005-05-19 Thread taro-x
2005-05-19 Kouji Sasaki <[EMAIL PROTECTED]> > > Added the feature to change row height in owner draw controls. > > The indentation looks a bit funky on this one -- can you please > use 4-space indentation? > > -- > Dimi. > I will send the modified patch with

Re: Corrected the determination of capturing inside EDIT_WM_MouseMove function.

2005-05-19 Thread taro-x
2005-05-19 Kouji Sasaki <[EMAIL PROTECTED]> > > Corrected the determination of capturing inside EDIT_WM_MouseMove > function. > > > - if (GetCapture() != es->hwndSelf) > > + if (GetCapture() != es->hwndSelf || !es->bCaptureState) > > Can we have > GetCapture() == e

Re: Added the callback item processing for LISTVIEW_EndEditLabelT function.

2005-05-19 Thread taro-x
2005-05-19 Kouji Sasaki <[EMAIL PROTECTED]> > > Added the callback item processing for LISTVIEW_EndEditLabelT function. > > I think this would be a bit more explicit and correct (is_textW() returns > false > if pszText == NULL) if the test is the other way around: > > +