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

Just a missing space in glossary page

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.6.1
    • 3.5.5, 3.6.3
    • Glossary
    • MOODLE_36_STABLE
    • MOODLE_35_STABLE, MOODLE_36_STABLE
    • MDL-64486-master
    • Hide

      Prerequisites:

      1. Create a course
      2. Create a glossary with the following:
        1. 'Entries shown per page' set to 1
        2. Contains atleast 5 glossary items

      Testing:

      1. Go to the glossary created and select 'All'
      2. Run the following in the browser inspector

        $('#page-mod-glossary-view #region-main br:first-of-type').hide()
        

      3. Confirm there is a space between "(Next)" and "All"
      Show
      Prerequisites: Create a course Create a glossary with the following: 'Entries shown per page' set to 1 Contains atleast 5 glossary items Testing: Go to the glossary created and select 'All' Run the following in the browser inspector $( '#page-mod-glossary-view #region-main br:first-of-type' ).hide() Confirm there is a space between "(Next)" and "All"

    Description

      When in a glossary in "Browse by alphabet" tab, paging links appear followed by "(Next)" and ALL which look like this:

      Page:  1  2  3  4  5  6  7  8  9  (Next)ALL

      A space is probably missing just after "(Next)" so that it would look like this:

      Page:  1  2  3  4  5  6  7  8  9  (NextALL

      An easy alternative would be to add a space just before word ALL in english fixes like this:

       ALL

      You can see the problem in the attached image and online there:

      https://moodle.org/mod/glossary/view.php?id=8542&mode=letter&hook=ALL&sortkey&sortorder=asc&fullsearch=0&page=-1

      but not there:

      https://moodle.org/mod/glossary/view.php?id=8542&mode=letter

       

      Attachments

        1. (space)ALL.png
          86 kB
          Stamos Sp.
        2. MDL-64486.jpg
          18 kB
          Anna Carissa Sadia

        Activity

          People

            peterdias Peter Dias
            stamos2 Stamos Sp.
            Mathew May Mathew May
            Sara Arjona (@sarjona) Sara Arjona (@sarjona)
            Anna Carissa Sadia Anna Carissa Sadia
            Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              11/Mar/19

              Time Tracking

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 10 minutes
                10m