Moodle

HTML too large for message compose window

Details

  • Type: Bug Bug
  • Status: Closed 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

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 - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.
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 - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

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

Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: