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

          Attachments

            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