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, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Adrian Greeve, Jake Dallimore, Mathew May, 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