Moodle
  1. Moodle
  2. MDL-40282

Collapsible editor loses border in IE8-9

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5.1
    • Component/s: HTML Editor (TinyMCE)
    • Labels:
      None
    • Testing Instructions:
      Hide

      Using Internet explorer 8 and/or 9.

      Open a question editing form and find a question with a collapsible editor. true/false > general > feedback fields for example.

      The attached screenshots show that the border of the collapsible editor should fit correctly the textarea. Instead it ends shortly to the right of the 'show editing tools' button.

      Show
      Using Internet explorer 8 and/or 9. Open a question editing form and find a question with a collapsible editor. true/false > general > feedback fields for example. The attached screenshots show that the border of the collapsible editor should fit correctly the textarea. Instead it ends shortly to the right of the 'show editing tools' button.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
      git@github.com:colchambers/moodle.git
    • Pull 2.5 Branch:
    • Pull Master Branch:
    • Rank:
      51050

      Description

      The tinymce text area border disappears on the right of the text area on IE 8 & 9. Attached an image to explain. At the top the feedback editor is highlight to show how the editor text area should appear.

      Below the correct response editor is highlighted showing the problem.

        Activity

        Hide
        Colin Chambers added a comment -

        The issue fixed

        Show
        Colin Chambers added a comment - The issue fixed
        Hide
        Colin Chambers added a comment -

        A very simple fix. Just cleared the editor table width. Copying the existing line clearing the height for a similar reason.

        Only affects IE 8&9. Not everyone with IE seems to have the problem.

        Show
        Colin Chambers added a comment - A very simple fix. Just cleared the editor table width. Copying the existing line clearing the height for a similar reason. Only affects IE 8&9. Not everyone with IE seems to have the problem.
        Hide
        Tim Hunt added a comment -

        +1 from me.

        Show
        Tim Hunt added a comment - +1 from me.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

        TIA and ciao

        Show
        Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
        Hide
        Colin Chambers added a comment -

        Thanks Eloy. Done.

        Show
        Colin Chambers added a comment - Thanks Eloy. Done.
        Hide
        Dan Poltawski added a comment -

        Integrated to master and 25, thanks Colin.

        (BTW I am sure I saw a duplicate of this somewhere, but searching I can't find it)

        Show
        Dan Poltawski added a comment - Integrated to master and 25, thanks Colin. (BTW I am sure I saw a duplicate of this somewhere, but searching I can't find it)
        Hide
        Rossiani Wijaya added a comment -

        This is working as expected.

        Tested for 2.5 and master for IE8 and 9.

        Test passed.

        Show
        Rossiani Wijaya added a comment - This is working as expected. Tested for 2.5 and master for IE8 and 9. Test passed.
        Hide
        Damyon Wiese added a comment -

        This issue is fixed! Hurray! Hurray!
        Your issue is fixed, what a wonderful day!

        Cheers, Damyon

        Show
        Damyon Wiese added a comment - This issue is fixed! Hurray! Hurray! Your issue is fixed, what a wonderful day! Cheers, Damyon

          People

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

            Dates

            • Created:
              Updated:
              Resolved: