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

          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: