Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-34503 META: Overhaul CANVAS theme
  3. MDL-34526

Add custommenu and block-region classes to CANVAS theme

          Details

          • Type: Sub-task
          • Status: Closed
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 2.3.1, 2.4
          • Fix Version/s: 2.4
          • Component/s: Themes
          • Labels:
          • Testing Instructions:
            Hide

            NB: Please ensure you have a custom menu set in Themes settings prior to testing.

            1. Select CANVAS theme using theme selector. (NOT by URL)
            2. Ensure AJAX & JavaScript are enabled.
            3. Turn editing on and TEST drag-n-drop works when moving blocks.
            4. With editing still enabled TEST docking and un-docking all blocks.
            5. Turn editing off then TEST docking and un-docking all blocks.
            6. TEST 3,4 & 5 in Home page, Course page and Report page for consistency
            7. Test to see that the Custom Menu is displayed and works correctly
            Show
            NB: Please ensure you have a custom menu set in Themes settings prior to testing. Select CANVAS theme using theme selector. (NOT by URL) Ensure AJAX & JavaScript are enabled. Turn editing on and TEST drag-n-drop works when moving blocks. With editing still enabled TEST docking and un-docking all blocks. Turn editing off then TEST docking and un-docking all blocks. TEST 3,4 & 5 in Home page, Course page and Report page for consistency Test to see that the Custom Menu is displayed and works correctly
          • Affected Branches:
            MOODLE_23_STABLE, MOODLE_24_STABLE
          • Fixed Branches:
            MOODLE_24_STABLE
          • Pull from Repository:
          • Pull Master Diff URL:

            Description

            block-region classes are a necessary requirement in layout files for themes to be able to work correctly when using AJAX drag-n-drop. At the moment CANVAS is missing these from region-post and region-pre in canvas/layout files.

            Also, like BASE theme, CANVAS should have the custommenu php in the layout files too, so that if CANVAS is copied to use as a base for new themes all the php is included that needs to be.

              Gliffy Diagrams

                Attachments

                  Issue Links

                  caused a regression

                  Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-35986 Serenity theme on 2.3.3 - custom menu not showing up

                  • Minor - Minor loss of function where workaround is possible
                  • Closed
                  has a non-specific relationship to

                  Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-34528 Remove redundent exclude stylesheet configurations from CANVAS theme

                  • Trivial - Cosmetic problem like misspelt words or misaligned text
                  • Closed

                  Improvement - An enhancement to an existing Moodle feature. MDL-32652 Make block drag-drop work throughout Moodle

                  • Major - Major loss of function, incorrect output
                  • Closed

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    lazydaisy Mary Evans added a comment -

                    I'm working on this now hoping to get it finished soon.

                    Show
                    lazydaisy Mary Evans added a comment - I'm working on this now hoping to get it finished soon.
                    Hide
                    Permalink
                    samhemelryk Sam Hemelryk added a comment -

                    Thanks Mary, this has been integrated now

                    Show
                    samhemelryk Sam Hemelryk added a comment - Thanks Mary, this has been integrated now
                    Hide
                    Permalink
                    andyjdavis Andrew Davis added a comment -

                    It all seems to work fine except for being able to drag and drop blocks on the course reports. I tried the participation report and activity completion. The non-ajax block moving works fine and dragging and dropping works fine on the site front page and the course page but I dont seem to be able to grab the blocks and move them around on the course reports.

                    I'm using the theme "leatherbound" as its based on canvas and I dont seem to be able to select canvas directly.

                    Show
                    andyjdavis Andrew Davis added a comment - It all seems to work fine except for being able to drag and drop blocks on the course reports. I tried the participation report and activity completion. The non-ajax block moving works fine and dragging and dropping works fine on the site front page and the course page but I dont seem to be able to grab the blocks and move them around on the course reports. I'm using the theme "leatherbound" as its based on canvas and I dont seem to be able to select canvas directly.
                    Hide
                    Permalink
                    poltawski Dan Poltawski added a comment -

                    Hi Andrew,

                    This is to be expected - there is an issue somewhere about this, the drag/drop only works on course pages at the moment. I think that means this can be passed.

                    Show
                    poltawski Dan Poltawski added a comment - Hi Andrew, This is to be expected - there is an issue somewhere about this, the drag/drop only works on course pages at the moment. I think that means this can be passed.
                    Hide
                    Permalink
                    poltawski Dan Poltawski added a comment -

                    The issue is MDL-32652

                    Show
                    poltawski Dan Poltawski added a comment - The issue is MDL-32652
                    Hide
                    Permalink
                    poltawski Dan Poltawski added a comment -

                    I'm passing this based on your comments Andrew. Please shout if i've got this wrong.

                    Show
                    poltawski Dan Poltawski added a comment - I'm passing this based on your comments Andrew. Please shout if i've got this wrong.
                    Hide
                    Permalink
                    poltawski Dan Poltawski added a comment -

                    asko, Дзякуй, ধন্যবাদ, Благодаря, Gràcies, 感谢, 謝謝, Hvala, Díky, Tak, Bedankt, Tänan, متشکریم, Salamat, Kiitokset, Merci, Grazas, Danke, Ευχαριστώ, આભાર, תודה, धन्यवाद, Köszönjük, Takk fyrir, Terima Kasih, Grazie, ありがとうございます, Рахмет, សូមអរគុណ, 감사합니다, gratiās, Pateicamies, Ačiū, Благодарам, Tēnā koa, Kia Ora Rawa Atu, आभारी आहोत, Талархал, Takk, Dziękuję, Obrigado, Mulţumesc, Engraziel, Спасибо, Fa'afetai, Хвала, Hvala, ස්තූතියි, Vďaka, Hvala, Mahadsanid, Thanks, Gracias, Tack, Salamat, நன்றி, నెనరులు, ขอบคุณค่ะ!

                    Your work has made it into this weeks Moodle release! There are no gold medals available this week - but millions around the world will benefit. Thank you!

                    Show
                    poltawski Dan Poltawski added a comment - asko, Дзякуй, ধন্যবাদ, Благодаря, Gràcies, 感谢, 謝謝, Hvala, Díky, Tak, Bedankt, Tänan, متشکریم, Salamat, Kiitokset, Merci, Grazas, Danke, Ευχαριστώ, આભાર, תודה, धन्यवाद, Köszönjük, Takk fyrir, Terima Kasih, Grazie, ありがとうございます, Рахмет, សូមអរគុណ, 감사합니다, gratiās, Pateicamies, Ačiū, Благодарам, Tēnā koa, Kia Ora Rawa Atu, आभारी आहोत, Талархал, Takk, Dziękuję, Obrigado, Mulţumesc, Engraziel, Спасибо, Fa'afetai, Хвала, Hvala, ස්තූතියි, Vďaka, Hvala, Mahadsanid, Thanks, Gracias, Tack, Salamat, நன்றி, నెనరులు, ขอบคุณค่ะ! Your work has made it into this weeks Moodle release! There are no gold medals available this week - but millions around the world will benefit. Thank you!

                      People

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

                        Dates

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