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

Just a missing space in glossary page

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.6.1
    • Fix Version/s: 3.5.5, 3.6.3
    • Component/s: Glossary
    • Labels:
    • Testing Instructions:
      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"
    • Affected Branches:
      MOODLE_36_STABLE
    • Fixed Branches:
      MOODLE_35_STABLE, MOODLE_36_STABLE
    • Pull from Repository:
    • Pull 3.5 Branch:
    • Pull Master Branch:
      MDL-64486-master

      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

          Activity

            People

            Assignee:
            peterdias Peter Dias
            Reporter:
            stamos2 Stamos Sp.
            Peer reviewer:
            Mathew May
            Integrator:
            Sara Arjona (@sarjona)
            Tester:
            Anna Carissa Sadia
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              11/Mar/19

                Time Tracking

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