Details

    • Testing Instructions:
      Hide

      1. Inspect the navigation block code
      2. Make sure that the not-expandable nodes contain an icon (in <img> tag) with an empty alt and title attribute.

      Show
      1. Inspect the navigation block code 2. Make sure that the not-expandable nodes contain an icon (in <img> tag) with an empty alt and title attribute.
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-30837-master
    • Rank:
      33837

      Description

      The custom bullets in the tree have the alt text of "Moodle" which is unnecessary and confusing.

      Potential Solution:
      The alt text should be an empty string.

        Activity

        Hide
        Rajesh Taneja added a comment -

        Code looks Good Fred,

        Pushing it for integration review.

        Show
        Rajesh Taneja added a comment - Code looks Good Fred, Pushing it for integration review.
        Hide
        Dan Poltawski added a comment -

        The main moodle.git repository has just 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
        Dan Poltawski added a comment - The main moodle.git repository has just 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
        Frédéric Massart added a comment -

        There you are. Rebased!

        Show
        Frédéric Massart added a comment - There you are. Rebased!
        Hide
        Sam Hemelryk added a comment -

        Thanks Fred, changes look spot on and this has been integrated now

        Show
        Sam Hemelryk added a comment - Thanks Fred, changes look spot on and this has been integrated now
        Hide
        Sam Hemelryk added a comment -

        Tested during integration review and passed

        Show
        Sam Hemelryk added a comment - Tested during integration review and passed
        Hide
        Eloy Lafuente (stronk7) added a comment -

        We could celebrate it today... but better if we perform a bigger party after releasing Moodle 2.3.

        Print this message and come to Perth that day, it's valid for one beer, wine, coke or... water, as you wish.

        Many thanks for your collaboration!

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - We could celebrate it today... but better if we perform a bigger party after releasing Moodle 2.3. Print this message and come to Perth that day, it's valid for one beer, wine, coke or... water, as you wish. Many thanks for your collaboration! Ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: