Moodle
  1. Moodle
  2. MDL-11242

HTML too large for message compose window

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: 1.8.2, 1.9
        • Fix Version/s: 1.9
        • Component/s: Messages
        • Labels:
          None
        • Affected Branches:
          MOODLE_18_STABLE, MOODLE_19_STABLE
        • Fixed Branches:
          MOODLE_19_STABLE
        • Rank:
          28591

          Description

          With HTML Editor on for messaging, the message compose window is too small to hold the HTML Area editor and the Send Message button,

          The HTMLArea widget is too wide, and overflows to the right. The send button appears too low on the frame, and requires scrolling down to press.

          Clearly visible on 1.8.x and HEAD (1.9). On Moodle v1.6 the sizes were correct – and quite pleasant to use.

          This is on Firefox 2.0.0.6 Windows.

            Issue Links

            has been marked as being related by

            Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-12935 Unable to enter description using IE7

            • Major - Major loss of function, incorrect output
            • Closed
            is duplicated by

            Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-12136 latest CVS upgrade has left me with HTML editor at 50% width

            • Minor - Minor loss of function where workaround is possible
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Martin Dougiamas added a comment -

              Yeah this has been annoying me too. I had a quick look into it but I couldn't make it smaller ... I wonder if it's related to XHTML or something like that.

              Show
              Martin Dougiamas added a comment - Yeah this has been annoying me too. I had a quick look into it but I couldn't make it smaller ... I wonder if it's related to XHTML or something like that.
              Hide
              Permalink
              Martin Dougiamas added a comment -

              OK, found it now and fixed in 1.9. Cheers!

              Show
              Martin Dougiamas added a comment - OK, found it now and fixed in 1.9. Cheers!
              Hide
              Permalink
              Martín Langhoff added a comment -

              This fix broke "fullscreen" htmlarea on IE6/7. Under IE the width is "100%" and we inconditionally append "px" to that.

              Comitting the fix into 18/19/HEAD...

              Show
              Martín Langhoff added a comment - This fix broke "fullscreen" htmlarea on IE6/7. Under IE the width is "100%" and we inconditionally append "px" to that. Comitting the fix into 18/19/HEAD...
              Hide
              Permalink
              Martín Langhoff added a comment -

              Committed - Now... to clean my hands after having touched the horrible MS Script Debugger!

              Show
              Martín Langhoff added a comment - Committed - Now... to clean my hands after having touched the horrible MS Script Debugger!
              Hide
              Permalink
              Stephen Bourget added a comment -

              Tested in both IE 6 and IE7
              Closing...

              Show
              Stephen Bourget added a comment - Tested in both IE 6 and IE7 Closing...
              Hide
              Permalink
              Ryan Smith added a comment -

              I just updated the 1.9 branch and this is fixed. Doesn't look like the fix was committed to the MOODLE_18_STABLE branch, thus, I'm still having the problem in Firefox.

              Show
              Ryan Smith added a comment - I just updated the 1.9 branch and this is fixed. Doesn't look like the fix was committed to the MOODLE_18_STABLE branch, thus, I'm still having the problem in Firefox.
              Hide
              Permalink
              Martín Langhoff added a comment -

              Oops! Sorry about that. The fix for the subsequent FF breakage is now on 18_STABLE too.

              If lib/editor/htmlarea/htmlarea.php v1.16.2.5 does not fix the problem for you, reopen this bug with a good steps-to-repro, so I can see it. As far as I could test today, with that fix htmlarea is working fine on FF v2.0 and IE6/7 both on the normal window and on the fullscreen.

              Show
              Martín Langhoff added a comment - Oops! Sorry about that. The fix for the subsequent FF breakage is now on 18_STABLE too. If lib/editor/htmlarea/htmlarea.php v1.16.2.5 does not fix the problem for you, reopen this bug with a good steps-to-repro, so I can see it. As far as I could test today, with that fix htmlarea is working fine on FF v2.0 and IE6/7 both on the normal window and on the fullscreen.
              Hide
              Permalink
              Ryan Smith added a comment -

              I just grabbed the latest from CVS and the issue is fixed for me on MOODLE_18_STABLE.

              Show
              Ryan Smith added a comment - I just grabbed the latest from CVS and the issue is fixed for me on MOODLE_18_STABLE.
              Hide
              Permalink
              Mathieu Petit-Clair added a comment -

              The change made in htmlarea.php breaks the htmlarea window in IE7/Vista.

              Show
              Mathieu Petit-Clair added a comment - The change made in htmlarea.php breaks the htmlarea window in IE7/Vista.
              Hide
              Permalink
              Mathieu Petit-Clair added a comment -

              I am reverting to the too-large window for 1.9, as the current setting breaks the htmleditor in every other case. This will also make the htmleditor show the same behavior in 1.8 and 1.9.

              Show
              Mathieu Petit-Clair added a comment - I am reverting to the too-large window for 1.9, as the current setting breaks the htmleditor in every other case. This will also make the htmleditor show the same behavior in 1.8 and 1.9.
              Hide
              Permalink
              Mathieu Petit-Clair added a comment -

              The fix for MDL-12726 seems to re-fix this.

              Show
              Mathieu Petit-Clair added a comment - The fix for MDL-12726 seems to re-fix this.

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: