[Mono-bugs] [Bug 358237] New: "richtext" variable is never set true for RichTextBox

[bugzilla_noreply at novell.com]

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


           Summary: "richtext" variable is never set true for RichTextBox
           Product: Mono: Class Libraries
           Version: SVN
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: P5 - None
         Component: Windows.Forms
        AssignedTo: mono-bugs at lists.ximian.com
        ReportedBy: jdpurcell2 at yahoo.com
         QAContact: mono-bugs at lists.ximian.com
          Found By: ---


TextBoxBase and TextControl check a variable called "richtext" in several
places but the value is always false because that's how it's initialized in
TextBoxBase's constructor and RichTextBox doesn't set it.

In TextBoxBase:
For SetBoundsCore and FixupHeight it would help fix some scaling problems if
the variable were set properly (or "this is RichTextBox" instead of
"richtext").  For FontOrColorChanged, I believe the "if" for richtext should be
removed, I think it broke some stuff when I changed text color.

In TextControl:
For MoveCaret PgUp it would help if the variable were set properly, but does
the rest of the code need to go in an "else"?  Same for PgDn.  Also for PgUp I
don't think viewport_y ever gets to 0 from pressing page up in a RichTextBox (I
think it does in TextBox though), it gets to 1 I guess (I forget).  I'm not
sure what needs to be done here really, but since the code inside the "if"
isn't active now anyway it could just be commented out until someone feels like
fixing it.


-- 
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.