Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-3348

MathJax isn't applied in glossary entries and forum posts

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.8.0
    • Fix Version/s: 3.8.1
    • Component/s: Filters
    • Labels:
    • Testing Instructions:
      Hide

      This issue only needs to be tested in 1 device.

      1. Login in master site and go to MM Dev > Glossary > A glossary (copy) [SF Case Number...
      2. Check that the MathJax in the term "Test fromulas" is rendered fine.
      3. Now go to MM Dev Testing > Forum> Standard forum for general use > formulas.
      4. Check that MathJax is rendered fine.
      Show
      This issue only needs to be tested in 1 device. Login in master site and go to MM Dev > Glossary > A glossary (copy) [SF Case Number... Check that the MathJax in the term "Test fromulas" is rendered fine. Now go to MM Dev Testing > Forum> Standard forum for general use > formulas. Check that MathJax is rendered fine.
    • Affected Branches:
      MOODLE_38_STABLE
    • Fixed Branches:
      MOODLE_38_STABLE
    • Sprint:
      Moodle App 3.8.1

      Description

      Reported via SalesForce.

      I took a look and the problem in glossary is that we first fetch the entry, and then we fetch the glossary. This means we don't have the contextInstanceId (glossary.coursemodule) when the entry is rendered, so filters aren't applied.

      The format-text directive should re-format the contents if the contextLevel or the contextInstanceId have changed. Also, templates might want to use an ngIf to prevent the contents to render before all the data is ready (for performance reasons).

      This is yet another issue caused because the elements in ng-contents (core-loading) are created before they're moved to the final destination, so we cannot force them to be created when the loaded variable is set to true.

        Attachments

          Activity

            People

            Assignee:
            dpalou Dani Palou
            Reporter:
            dpalou Dani Palou
            Peer reviewer:
            Pau Ferrer
            Integrator:
            Juan Leyva
            Tester:
            Isabel Renedo Rouco
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              25/Mar/20