Moodle
  1. Moodle
  2. MDL-34405

Formal white quiz overview background missing

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.3.1
        • Fix Version/s: 2.2.5, 2.3.2
        • Component/s: Quiz, Themes
        • Labels:
        • Rank:
          42803

          Description

          When viewing mod/quiz/report.php the formal white theme misses the white background.
          See screenshot.

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Tim Hunt added a comment -

            Formal white issue, so -> Daniele.

            Show
            Tim Hunt added a comment - Formal white issue, so -> Daniele.
            Hide
            Permalink
            Daniele Cordella added a comment -

            Hi all and thanks Robin and Tim.
            I added the missing background since m21.
            AFAIK, all is ready for a peer review.
            Let me know your opinion.
            Ciao.

            Show
            Daniele Cordella added a comment - Hi all and thanks Robin and Tim. I added the missing background since m21. AFAIK, all is ready for a peer review. Let me know your opinion. Ciao.
            Hide
            Permalink
            Daniele Cordella added a comment -

            If someone can explain me why the "Pull 2.3 Diff URL:" link must be different from all the other to work fine, I would be grateful.

            Show
            Daniele Cordella added a comment - If someone can explain me why the "Pull 2.3 Diff URL:" link must be different from all the other to work fine, I would be grateful.
            Hide
            Permalink
            Mary Evans added a comment -

            Daniele
            You need to add a new branch for MOODLE_23_STABLE to your GITHUB

            At the moment you only have MOODLE_21_STABLE and MOODLE_22_STABLE

            Check this forum discussion I had same problem last year...
            http://moodle.org/mod/forum/discuss.php?d=192524

            Show
            Mary Evans added a comment - Daniele You need to add a new branch for MOODLE_23_STABLE to your GITHUB At the moment you only have MOODLE_21_STABLE and MOODLE_22_STABLE Check this forum discussion I had same problem last year... http://moodle.org/mod/forum/discuss.php?d=192524
            Hide
            Permalink
            Daniele Cordella added a comment -

            Thanks Mary.
            You are right. You are really 100% right!
            I forget to create it to my github when MOODLE_23_stable was born.
            Thank you very much!

            Show
            Daniele Cordella added a comment - Thanks Mary. You are right. You are really 100% right! I forget to create it to my github when MOODLE_23_stable was born. Thank you very much!
            Hide
            Permalink
            Tim Hunt added a comment -

            +1 looks good to me. Submitting for integration.

            Show
            Tim Hunt added a comment - +1 looks good to me. Submitting for integration.
            Hide
            Permalink
            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
            Permalink
            Dan Poltawski added a comment -

            Thanks Daniele, i've integrated this to 22, 23 and master. (We are only supporting security fixes in 21 now).

            Show
            Dan Poltawski added a comment - Thanks Daniele, i've integrated this to 22, 23 and master. (We are only supporting security fixes in 21 now).
            Hide
            Permalink
            Andrew Davis added a comment -

            Looks good. Passing.

            Show
            Andrew Davis added a comment - Looks good. Passing.
            Hide
            Permalink
            Eloy Lafuente (stronk7) added a comment -

            Fixed STOP Closed STOP Thanks STOP

            Yay, imagination! Ciao

            Show
            Eloy Lafuente (stronk7) added a comment - Fixed STOP Closed STOP Thanks STOP Yay, imagination! Ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: