richedit: text that does not need scrollbar should also result in a scroll range of 0. Tests for this behavior.

[Alex Villací­s Lasso]

Eric Pouech escribió:
> Alex Villací­s Lasso a écrit :
>   
>> Even though the code freeze is still in effect, I post this so that it 
>> will be reviewed. For more information, see bug #12311.
>>
>> Changelog:
>> * richedit: empty text should result in a scroll range of 0.
>> * Tests for this behavior.
>>
>> ------------------------------------------------------------------------
>>
>>
>>     
> what I don't understand is why the height of an empty doc is not zero (I 
> had similar issues with the height of a doc where we systematically get 
> one row too much). I believe this root cause should be fixed instead of 
> the band aid your patch is providing
> A+
>
>   
The height of the document is irrelevant. Instead, if the actual height 
is less than the height of the client area, the scrollbar should also be 
set to zero, as shown by tests included in the attached patch. Empty 
text is just one special case.

Changelog:
* Text that does not need to be scrolled should also result in a scroll 
range of zero
* Tests for this behavior

-- 
perl -e '$x=2.4;print sprintf("%.0f + %.0f = %.0f\n",$x,$x,$x+$x);'

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0011-richedit-nonscrolled-text.patch
Type: text/x-patch
Size: 1382 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20080514/ba3d100b/attachment.bin