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:

      Description

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

        Gliffy Diagrams

          Activity

          Hide
          Tim Hunt added a comment -

          Formal white issue, so -> Daniele.

          Show
          Tim Hunt added a comment - Formal white issue, so -> Daniele.
          Hide
          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
          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
          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
          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
          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
          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
          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
          Andrew Davis added a comment -

          Looks good. Passing.

          Show
          Andrew Davis added a comment - Looks good. Passing.
          Hide
          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: