Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-9646

htmlarea - if textarea is nested with in it's label, htmlarea adds form elements within the label

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Won't Fix
    • Affects Version/s: 1.8
    • Fix Version/s: None
    • Component/s: HTML Editor (TinyMCE)
    • Labels:
      None
    • Affected Branches:
      MOODLE_18_STABLE

      Description

      if the textarea is nested within it's label, htmlarea generates other form elements within that label (invalid HTML as well as buggy in Firefox). Am going to commit the following fix at some point today -

      LINE 643

      if (textarea.parentNode.nodeName.toLowerCase()=='label')

      { textarea.parentNode.parentNode.insertBefore(htmlarea,textarea.parentNode); }

      else

      { textarea.parentNode.insertBefore(htmlarea, textarea); }

      ...replaces...

      textarea.parentNode.insertBefore(htmlarea, textarea);

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            aforth aforth added a comment -

            Committed fix to Moodle head.

            Show
            aforth aforth added a comment - Committed fix to Moodle head.
            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting this issue.

            We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported.

            If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed.

            Michael d;

            lqjjLKA0p6

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting this issue. We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported. If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed. Michael d; lqjjLKA0p6
            Hide
            salvetore Michael de Raadt added a comment -

            I'm closing this issue as it has become inactive and does not appear to affect a current supported version. If you are encountering this problem or one similar, please launch a new issue.

            Show
            salvetore Michael de Raadt added a comment - I'm closing this issue as it has become inactive and does not appear to affect a current supported version. If you are encountering this problem or one similar, please launch a new issue.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: