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

Make Bootstrapbase breadcrumb seperator respect $THEME->rarrow

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • 2.5, 2.5.4, 2.6.1
    • 2.5.5, 2.6.2
    • HTML and CSS, Themes
    • MOODLE_25_STABLE, MOODLE_26_STABLE
    • MOODLE_25_STABLE, MOODLE_26_STABLE
    • 40693-27
    • Hide
      1. Log in as admin
      2. Change to the clean theme.
      3. Browse to a course
      4. Observe that ► is used to separate navbar items.
      5. Change to a RTL lang and ensure the separator is now ◄
      6. Edit the theme/clean/config.php and set the $THEME->rarrow = '/' and $THEME->larrow = '\';
      7. Retest the above and check you get / and \ as your separators now.
      Show
      Log in as admin Change to the clean theme. Browse to a course Observe that ► is used to separate navbar items. Change to a RTL lang and ensure the separator is now ◄ Edit the theme/clean/config.php and set the $THEME->rarrow = '/' and $THEME->larrow = '\'; Retest the above and check you get / and \ as your separators now.
    • 3
    • FRONTEND Sprint 8

    Description

      The breadcrumb seperator "/" is hardcoded in a renderer for the Bootstrapbase/Clean themes, here:

      https://github.com/moodle/moodle/blob/master/theme/bootstrapbase/renderers/core_renderer.php#L61

      It should probably get the character from $THEME->rarrow so that child themes have an easy way to change this character.

      (A better way to fix this is to do it all in CSS, like Bootstrap v3 or the Pure YUI framework, but the actual character is used in Bootstrap 2.3 for IE7 compatability and it's what Moodle themers are used to).

      Attachments

        Issue Links

          Activity

            People

              samhemelryk Sam Hemelryk
              bawjaws David Scotson
              Frédéric Massart Frédéric Massart
              Dan Poltawski Dan Poltawski
              Jérôme Mouneyrac Jérôme Mouneyrac
              Amy Groshek, David Scotson, David Woloszyn, Huong Nguyen, Jake Dallimore, Michael Hawkins, Stevani Andolo, Huong Nguyen, Bas Brands, Mathew May
              Votes:
              1 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                10/Mar/14