https://bugzilla.novell.com/show_bug.cgi?id=440220

User [EMAIL PROTECTED] added comment
https://bugzilla.novell.com/show_bug.cgi?id=440220#c3


Andy Hume <[EMAIL PROTECTED]> changed:

           What    |Removed                                         |Added
----------------------------------------------------------------------------
            Summary|Multiple faults in ProgressBar: ForeColor,      |[PATCH] 
Multiple faults in ProgressBar:
                   |negative Step, missing/wrong exceptions         |ForeColor, 
negative Step, missing/wrong
                   |                                                |exceptions




--- Comment #3 from Andy Hume <[EMAIL PROTECTED]>  2008-10-30 09:01:15 MDT ---
Attached patch fixes 1, 2, 3, 4, 5, 7, 8.  I haven't looked at the bad
calculation of #6, Marquee speed.

The changes to ProgressBar.cs should be uncontroversial.

The changes to Theme**.cs include possibly more than necessary.  Firstly
removing progressbarblock_color for #1 is simple.  Fix to #7 actually only
needs however:
+  block_width = Math.Max (block_width, 0); // block_width is used to break out
the loop below, it must be +ve!

The root cause is that ProgressBar.UpdateAreas doesn't ensure non-negative,
would that affect anything else?


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to