Details

    • Testing Instructions:
      1. Login as admin
      2. Purge all caches
      3. Turn on editing mode
      4. Check the roles icon is the same size as the others on the blocks
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull from Repository:

      Description

      the roles.gif icon has an other size athn other icons used on course page:
      roles.gif = 16x16 px
      edit.gif = 11x11 px

      All icons should have the same size. I suggest generally 16x16. 11x11 is much to small.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            madmax108 Navin Pai added a comment -

            Just browsing around, I found that there's a 16x16 'edit.gif' icon already present in Moodle. M attaching a patch that I think should fix the error, and display a 16x16 icon instead of the 11x11 icon

            Show
            madmax108 Navin Pai added a comment - Just browsing around, I found that there's a 16x16 'edit.gif' icon already present in Moodle. M attaching a patch that I think should fix the error, and display a 16x16 icon instead of the 11x11 icon
            Hide
            madmax108 Navin Pai added a comment -

            Have simply replaced /t/edit.gif by/i/edit.gif ...which is the path of the 16x16 icon

            Change made in

            {install_dir}

            /mod/glossary/settings.php

            Show
            madmax108 Navin Pai added a comment - Have simply replaced /t/edit.gif by/i/edit.gif ...which is the path of the 16x16 icon Change made in {install_dir} /mod/glossary/settings.php
            Hide
            tsala Helen Foster added a comment -

            I'd not noticed before now that the roles icon is bigger than the others! All the icons next to course activities are 11x11. I'm not sure whether 16x16 would work there.

            Navin, thanks for your patch.

            Show
            tsala Helen Foster added a comment - I'd not noticed before now that the roles icon is bigger than the others! All the icons next to course activities are 11x11. I'm not sure whether 16x16 would work there. Navin, thanks for your patch.
            Hide
            korpelainen Mauno Korpelainen added a comment -

            The main problem is not in settings of glossary but in lib/blocklib.php and line 990 there which is using

            $controls[] = array('url' => $CFG->wwwroot . '/' . $CFG->admin .
            '/roles/assign.php?contextid=' . $block->context->id . '&returnurl=' . urlencode($return),
            'icon' => 'i/roles', 'caption' => get_string('assignroles', 'role'));

            For some reason folder pix/t does not have a 11x11px roles.gif icon and blocklib is using 16x16px icon from pix/i

            To make all control icons equal size we should add a new 11x11px roles.gif icon to folder pix/t and change the previous code to

            $controls[] = array('url' => $CFG->wwwroot . '/' . $CFG->admin .
            '/roles/assign.php?contextid=' . $block->context->id . '&returnurl=' . urlencode($return),
            'icon' => 't/roles', 'caption' => get_string('assignroles', 'role'));

            If some theme needs to use larger control icons you just need to add all the necessary larger icons (for example 16x16px) to theme/yourtheme/pix_core/t - it's so simple...

            Show
            korpelainen Mauno Korpelainen added a comment - The main problem is not in settings of glossary but in lib/blocklib.php and line 990 there which is using $controls[] = array('url' => $CFG->wwwroot . '/' . $CFG->admin . '/roles/assign.php?contextid=' . $block->context->id . '&returnurl=' . urlencode($return), 'icon' => 'i/roles', 'caption' => get_string('assignroles', 'role')); For some reason folder pix/t does not have a 11x11px roles.gif icon and blocklib is using 16x16px icon from pix/i To make all control icons equal size we should add a new 11x11px roles.gif icon to folder pix/t and change the previous code to $controls[] = array('url' => $CFG->wwwroot . '/' . $CFG->admin . '/roles/assign.php?contextid=' . $block->context->id . '&returnurl=' . urlencode($return), 'icon' => 't/roles', 'caption' => get_string('assignroles', 'role')); If some theme needs to use larger control icons you just need to add all the necessary larger icons (for example 16x16px) to theme/yourtheme/pix_core/t - it's so simple...
            Hide
            phalacee Jason Fowler added a comment -

            I'll be fixing this issue for 2.1 ... it seems to have been fixed for 2.3dev already, I haven't checked 2.2 yet though

            Show
            phalacee Jason Fowler added a comment - I'll be fixing this issue for 2.1 ... it seems to have been fixed for 2.3dev already, I haven't checked 2.2 yet though
            Hide
            phalacee Jason Fowler added a comment -

            The bug only exists for 2.1 so I will fix for that, 2.0 has been dropped for general bug support

            Show
            phalacee Jason Fowler added a comment - The bug only exists for 2.1 so I will fix for that, 2.0 has been dropped for general bug support
            Hide
            phalacee Jason Fowler added a comment -

            Code checked in, simple one-line CSS fix, should be really easy for anyone using 2.0 to patch their instances too.

            Show
            phalacee Jason Fowler added a comment - Code checked in, simple one-line CSS fix, should be really easy for anyone using 2.0 to patch their instances too.
            Hide
            poltawski Dan Poltawski added a comment -

            Makes sense.

            If I were nitpicking i'd match the same order as it is in 2.2 to indicate that its the same (also I prefer the height/width being next to each other) but that is of course irrelevant!

            Show
            poltawski Dan Poltawski added a comment - Makes sense. If I were nitpicking i'd match the same order as it is in 2.2 to indicate that its the same (also I prefer the height/width being next to each other) but that is of course irrelevant!
            Hide
            phalacee Jason Fowler added a comment -

            I think it's fine the way it is, pushing this through for integration

            Show
            phalacee Jason Fowler added a comment - I think it's fine the way it is, pushing this through for integration
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Some hours ago...

            the main moodle.git repository has been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Some hours ago... the main moodle.git repository has been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            phalacee Jason Fowler added a comment -

            rebased the branch

            Show
            phalacee Jason Fowler added a comment - rebased the branch
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Jason this has been integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Jason this has been integrated now
            Hide
            phalacee Jason Fowler added a comment -

            Thanks Sam

            Show
            phalacee Jason Fowler added a comment - Thanks Sam
            Hide
            salvetore Michael de Raadt added a comment -

            I just added a step to purge caches in the testing instructions.

            Show
            salvetore Michael de Raadt added a comment - I just added a step to purge caches in the testing instructions.
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success!

            Show
            salvetore Michael de Raadt added a comment - Test result: Success!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            FCT (fixed, closing, thanks). Ciao

            "I feel a very unusual sensation - if it is not indigestion, I think it must be gratitude!"
            ~ Benjamin Disraeli

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - FCT (fixed, closing, thanks). Ciao "I feel a very unusual sensation - if it is not indigestion, I think it must be gratitude!" ~ Benjamin Disraeli

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/12