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

The Glossary summary is wrapped by a <p> tag when viewing it but it should be wrapped by a <div> tag

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.3
    • Fix Version/s: 1.9.4, 2.0
    • Component/s: Accessibility, Glossary
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE

      Description

      When creating a glossary, in its summary, people tend to use tags such as <p>, <ul>, <ol> or even <div> because they want to create paragraphs. The problem is that when someone watches a glossary, its summary is wrapped with the <p> tag, which doesnt allow does tags inside. Example of a normal generated summary:
      <div id="intro" class="generalbox box">
      <p>
      This is a glossary test
      </p>
      </div>

      Note that "This glossary test" was written without the <p> tag in the summary editor, but if we want to create two paragrapas in the summary, such as:
      <p>This is glossary test</p>
      <p>This are its specifications</p>

      This will be translated to:
      <div id="intro" class="generalbox box"><p>
      <p>This is glossary test</p>
      <p>This are its specifications</p>
      </p</div>

      Which is, indeed, wrong.

      The easiest solution is to change the outer <p> for a <div> tag in the code.

        Attachments

          Activity

            People

            • Assignee:
              dongsheng Dongsheng Cai
              Reporter:
              david.horat David Horat
              Tester:
              David Horat
              Participants:
              Component watchers:
              Adrian Greeve, Mihail Geshoski, Peter Dias, Adrian Greeve, Mihail Geshoski, Peter Dias
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                28/Jan/09