Re: comctl32: correct handling of toolbar separators' size

2009-03-18 Thread Rob Shearman
2009/3/18 Igor Tarasov : > 2009/3/18 Rob Shearman : >> 2009/3/18 Igor Tarasov : Please ensure that there are no regressions in the display of IE6's toolbars after this patch. This is the main user of undocumented toolbar features. > > Installed IE6, tested both version (with my patch

Re: comctl32: correct handling of toolbar separators' size

2009-03-18 Thread Igor Tarasov
2009/3/18 Rob Shearman : > 2009/3/18 Igor Tarasov : >>> Please ensure that there are no regressions in the display of IE6's >>> toolbars after this patch. This is the main user of undocumented >>> toolbar features. Installed IE6, tested both version (with my patch and without it). Image is pixel-p

Re: comctl32: correct handling of toolbar separators' size

2009-03-18 Thread Rob Shearman
2009/3/18 Igor Tarasov : >> Please ensure that there are no regressions in the display of IE6's >> toolbars after this patch. This is the main user of undocumented >> toolbar features. > > How can I do that? Follow the how-to here: http://appdb.winehq.org/objectManager.php?sClass=version&iId=469

Re: comctl32: correct handling of toolbar separators' size

2009-03-18 Thread Igor Tarasov
> Please ensure that there are no regressions in the display of IE6's > toolbars after this patch. This is the main user of undocumented > toolbar features. How can I do that? >> -       btnPtr->cx = lptbbi->cx; >> +        btnPtr->cx = lptbbi->cx; > Whitespace only changes shouldn't be included

Re: comctl32: correct handling of toolbar separators' size

2009-03-18 Thread Rob Shearman
2009/3/17 Igor Tarasov : > Currently, wine uses iBitmap property for determining toolbar > separators width all the time, stating it's an undocumented feature. > This is not correct. > > According to MSDN, iBitmap is used only on inserting separator (since > there is no cx field in TBBUTTON structu