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

There's no svg content bank icon for themes don't use fontawesome

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Testing for Boost theme:

      1. Delete this line on theme/boost/config.php file:

        $THEME->iconsystem = \core\output\icon_system::FONTAWESOME;

      2. Log in as admin, using Boost theme.
      3. Go to Dashboard
      4. Confirm the given icon is rendered for the drawer Content bank element.

      Testing for Classic theme:

      1. Delete this line on theme/classic/config.php file:

        $THEME->iconsystem = '\\theme_classic\\output\\icon_system_fontawesome';

      2. Log in as admin, using Classic theme.
      3. Open Site pages on 'Navigation block'
      4. Confirm the given icon is rendered for the Content bank element.

       

      Show
      Testing for Boost theme: Delete this line on theme/boost/config.php file: $THEME->iconsystem = \core\output\icon_system::FONTAWESOME; Log in as admin, using Boost theme. Go to Dashboard Confirm the given icon is rendered for the drawer Content bank element. Testing for Classic theme: Delete this line on theme/classic/config.php file: $THEME->iconsystem = '\\theme_classic\\output\\icon_system_fontawesome'; Log in as admin, using Classic theme. Open Site pages on 'Navigation block' Confirm the given icon is rendered for the  Content bank element.  
    • Affected Branches:
      MOODLE_39_STABLE
    • Fixed Branches:
      MOODLE_39_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-68614-master

      Description

      We should create an svg formatted icon called contentbank, and add it to pix/i directory. Themes don't use font awesome will render the given icon.

        Attachments

        1. MDL-68614.jpg
          MDL-68614.jpg
          19 kB
        2. paint-brush.png
          paint-brush.png
          0.4 kB
        3. paint-brush.svg
          0.5 kB

          Activity

            People

            Assignee:
            amaia Amaia Anabitarte
            Reporter:
            amaia Amaia Anabitarte
            Peer reviewer:
            Neill Magill
            Integrator:
            Eloy Lafuente (stronk7)
            Tester:
            Anna Carissa Sadia
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              15/Jun/20

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour, 46 minutes
                1h 46m