Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-33130 Dialogue unification project for 2.3
  3. MDL-33482

Tidy up activity chooser to give it more of a filepicker style

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Course
    • Labels:

      Description

      Smoother curves, shadows and fonts.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              dougiamas Martin Dougiamas added a comment -

              Code moved here from parent.

              Show
              dougiamas Martin Dougiamas added a comment - Code moved here from parent.
              Hide
              barbararamiro Barbara Ramiro added a comment - - edited

              Andrew, when .modchooser was changed to .chooserdiagloge some styles didnt work in IE so I needed to add !important. Well i could be wrong but thats the first thing I noticed. I wonder if Marina's comment on MDL-32764 could be applied here to avoid using !important. Also please read the comment here https://github.com/barbararamiro/moodle/commit/5e2843481167ab0c865fed35fbdad0dcb2a14184.

              Show
              barbararamiro Barbara Ramiro added a comment - - edited Andrew, when .modchooser was changed to .chooserdiagloge some styles didnt work in IE so I needed to add !important. Well i could be wrong but thats the first thing I noticed. I wonder if Marina's comment on MDL-32764 could be applied here to avoid using !important. Also please read the comment here https://github.com/barbararamiro/moodle/commit/5e2843481167ab0c865fed35fbdad0dcb2a14184 .
              Hide
              poltawski Dan Poltawski added a comment -

              I have integrated this.

              There is the !important issue, but I am going to create a new issue for that so that we can get this nice style out there and tested.

              Show
              poltawski Dan Poltawski added a comment - I have integrated this. There is the !important issue, but I am going to create a new issue for that so that we can get this nice style out there and tested.
              Hide
              poltawski Dan Poltawski added a comment -

              The !important issue is MDL-33584 (and don't be fooled, it really is important )

              Show
              poltawski Dan Poltawski added a comment - The !important issue is MDL-33584 (and don't be fooled, it really is important )
              Hide
              poltawski Dan Poltawski added a comment -

              Attaching screenshot of this in IE9. Note the imperfect corners and border around it.

              Show
              poltawski Dan Poltawski added a comment - Attaching screenshot of this in IE9. Note the imperfect corners and border around it.
              Hide
              poltawski Dan Poltawski added a comment -

              I tested this on Firefox, Chrome and Safari where it looked the same and perefect.

              On IE9 there were slight imperfections as i've just uploaded in screenshot. I think these are relatively minor though, so passing this.

              Show
              poltawski Dan Poltawski added a comment - I tested this on Firefox, Chrome and Safari where it looked the same and perefect. On IE9 there were slight imperfections as i've just uploaded in screenshot. I think these are relatively minor though, so passing this.
              Hide
              barbararamiro Barbara Ramiro added a comment -

              Dan, imperfection removed with !important. I know you hate it but it will eventually be removed.

              Show
              barbararamiro Barbara Ramiro added a comment - Dan, imperfection removed with !important. I know you hate it but it will eventually be removed.
              Hide
              fred Frédéric Massart added a comment -

              Discovered a little issue when enabling Assignment 2.2

              Show
              fred Frédéric Massart added a comment - Discovered a little issue when enabling Assignment 2.2
              Hide
              dobedobedoh Andrew Nicols added a comment -

              Assume that you mean the radio buttons for assignment 2.2 are too far to the left (agreed). This issue should probably be assigned to Barbara rather than myself - she's been handling the theming side of things.

              Show
              dobedobedoh Andrew Nicols added a comment - Assume that you mean the radio buttons for assignment 2.2 are too far to the left (agreed). This issue should probably be assigned to Barbara rather than myself - she's been handling the theming side of things.
              Hide
              poltawski Dan Poltawski added a comment -

              Fred - please file a new issue.

              Show
              poltawski Dan Poltawski added a comment - Fred - please file a new issue.
              Hide
              stronk7 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
              stronk7 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
              Hide
              poltawski Dan Poltawski added a comment -

              I'm creating a new issue to integrate the extra commit which didn't get integrated fixing IE

              Show
              poltawski Dan Poltawski added a comment - I'm creating a new issue to integrate the extra commit which didn't get integrated fixing IE
              Hide
              lazydaisy Mary Evans added a comment -

              @Barbara

              I would just like to query two of the rules set in the CSS changes to theme/base/core.css

              .moodle-dialogue-base .moodle-dialogue {
                  background-color: transparent!important;
                  border: 0px solid transparent!important;
              }

              Firstly, TRANSPARENT is NOT a color, it is the DEFAULT setting for the BACKGROUND property ONLY, so adding !important in this context is not necessary, indeed it is to be avoided.

              Secondly, since TRANSPARENT is not a color, border: 0 solid transparent!important is not valid, since a border with NO width does not exist, so has not need for a style nor a color.

              So the above rules would be better written like so...

              .moodle-dialogue-base .moodle-dialogue {
                  background: none;
                  border: 0 none;
              }

              Show
              lazydaisy Mary Evans added a comment - @Barbara I would just like to query two of the rules set in the CSS changes to theme/base/core.css .moodle-dialogue-base .moodle-dialogue { background-color: transparent!important; border: 0px solid transparent!important; } Firstly, TRANSPARENT is NOT a color, it is the DEFAULT setting for the BACKGROUND property ONLY, so adding !important in this context is not necessary, indeed it is to be avoided. Secondly, since TRANSPARENT is not a color, border: 0 solid transparent!important is not valid, since a border with NO width does not exist, so has not need for a style nor a color. So the above rules would be better written like so... .moodle-dialogue-base .moodle-dialogue { background: none; border: 0 none; }
              Hide
              barbararamiro Barbara Ramiro added a comment -

              Thanks Mary, it's been revised under MDL-33661 but I can't make it work on IE9 without !important.

              Show
              barbararamiro Barbara Ramiro added a comment - Thanks Mary, it's been revised under MDL-33661 but I can't make it work on IE9 without !important.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    25/Jun/12