Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.3
    • Fix Version/s: 2.4
    • Component/s: Usability
    • Labels:
      None
    • Testing Instructions:
      Hide
      1. Navigate to places where:
        • Elements can be moved up and down (sometimes only available when JS is off, IE: section moving)
        • Elements can be sorted in a table (users list, ...)
      2. Make sure
        • the new icon are used
        • they look good (margin, padding, vertical-align)
        • they make sense

      FYI: Few places to look:

      1. Question bank
      2. admin pages with tables (Browse users)
      3. Course enrolment page
      4. Course edit page without ajax or JS.
      Show
      Navigate to places where: Elements can be moved up and down (sometimes only available when JS is off, IE: section moving) Elements can be sorted in a table (users list, ...) Make sure the new icon are used they look good (margin, padding, vertical-align) they make sense FYI: Few places to look: Question bank admin pages with tables (Browse users) Course enrolment page Course edit page without ajax or JS.
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-36415-master-int

      Description

      • Go to a course enrolment page
      • The arrow to order users by last name ascending/descending is larger than before, probably too large

      See attached screenshots

        Gliffy Diagrams

        1. arrowafter.png
          4 kB
        2. arrowbefore.png
          5 kB
        3. Edit questions_up arrow.png
          9 kB

          Issue Links

            Activity

            Hide
            fred Frédéric Massart added a comment -

            This might need a fix in the code as well to point to another icon in such case.

            Show
            fred Frédéric Massart added a comment - This might need a fix in the code as well to point to another icon in such case.
            Hide
            barbararamiro Barbara Ramiro added a comment -

            Assigning this to you Fred

            Show
            barbararamiro Barbara Ramiro added a comment - Assigning this to you Fred
            Hide
            fred Frédéric Massart added a comment -

            Yay! \o/

            Show
            fred Frédéric Massart added a comment - Yay! \o/
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Thanks Fred,

            Patch looks Great, although as discussed up arrow is not aligned well.

            Adding a screen-shot for reference.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Thanks Fred, Patch looks Great, although as discussed up arrow is not aligned well. Adding a screen-shot for reference.
            Hide
            fred Frédéric Massart added a comment -

            Thanks Raj, the look and feel was not as great as it could have been. I have created a new selector specific for this kind of icon, and add some rules to it. It now looks better. I also forgot about the upgrade.txt file which I have updated now. Pushing for integration.

            Cheers!

            Show
            fred Frédéric Massart added a comment - Thanks Raj, the look and feel was not as great as it could have been. I have created a new selector specific for this kind of icon, and add some rules to it. It now looks better. I also forgot about the upgrade.txt file which I have updated now. Pushing for integration. Cheers!
            Hide
            poltawski Dan Poltawski added a comment -

            Integrated, thanks

            Show
            poltawski Dan Poltawski added a comment - Integrated, thanks
            Hide
            markn Mark Nelson added a comment -

            Now that's some shmexy looking icons there if I do say so myself. Passing this, thanks Fredos.

            Show
            markn Mark Nelson added a comment - Now that's some shmexy looking icons there if I do say so myself. Passing this, thanks Fredos.
            Hide
            poltawski Dan Poltawski added a comment -

            Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week!

            ciao
            Dan

            Show
            poltawski Dan Poltawski added a comment - Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week! ciao Dan

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  3/Dec/12