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 Bug
    • Status: Closed
    • Priority: Minor 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
    • Rank:
      3599

      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);

        Activity

        Hide
        aforth added a comment -

        Committed fix to Moodle head.

        Show
        aforth added a comment - Committed fix to Moodle head.
        Hide
        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
        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
        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
        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: