Moodle
  1. Moodle
  2. MDL-3625

$menu in header does too many things

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Inactive
    • Affects Version/s: 1.5, 1.6.9, 1.7.7, 1.8.8, 1.9.4
    • Fix Version/s: None
    • Component/s: Themes, Usability
    • Labels:
      None
    • Environment:
      All
    • Affected Branches:
      MOODLE_15_STABLE, MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      The $menu variable in the themes header.html does too many things.

      On the front page of your Moodle it tells you who you are logged in as as well as allowing you to choose a language (as long as that option is switched on)

      On Course index pages it only tells you if you are logged in.

      On Resource pages it gives you a button for viewing logs (again, only if switched on), a jump menu, and a previous and next button.

      All of these things are pretty cool, but it's a bit of straightjacket that you can only specify them to be in one place and, by extension surround them with a single bit of HTML and CSS style.

      You may well want all of the above to appear but in different places and styled differently. Or you may only want some elements e.g. the jump menu without the backwards and forwards arrows.

        Gliffy Diagrams

        1. header_course.png
          117 kB
        2. header_database.png
          100 kB
        3. header_site.png
          95 kB

          Issue Links

            Activity

            Hide
            Martin Dougiamas added a comment -

            From Urs Hunkler (u.hunkler at elearningkontor.de) Friday, 15 July 2005, 04:39 AM:

            Martin, David made a good proposal. What do you think? Can you divide the different functions?

            From Martin Dougiamas (martin at moodle.com) Friday, 15 July 2005, 11:54 AM:

            I'm very aware of the problem and I totally agree, but it's a legacy problem that will require a lot of changes around Moodle (in many print_header calls) ... we probably should have tackled it in 1.5 but we didn't ...

            To start with can you look at it from a CSS point of view and come up with a good naming scheme for all these bits and pieces?

            Show
            Martin Dougiamas added a comment - From Urs Hunkler (u.hunkler at elearningkontor.de) Friday, 15 July 2005, 04:39 AM: Martin, David made a good proposal. What do you think? Can you divide the different functions? From Martin Dougiamas (martin at moodle.com) Friday, 15 July 2005, 11:54 AM: I'm very aware of the problem and I totally agree, but it's a legacy problem that will require a lot of changes around Moodle (in many print_header calls) ... we probably should have tackled it in 1.5 but we didn't ... To start with can you look at it from a CSS point of view and come up with a good naming scheme for all these bits and pieces?
            Hide
            Urs Hunkler added a comment -

            Still not solved.

            Show
            Urs Hunkler added a comment - Still not solved.
            Hide
            Anthony Borrow added a comment -

            Moodle.org is consistent in that it displays a constant header including logininfo and chooselang menu. I began looking at the standard theme and saw that it was very inconsistent as described. I'll attach some screenshots.

            Show
            Anthony Borrow added a comment - Moodle.org is consistent in that it displays a constant header including logininfo and chooselang menu. I began looking at the standard theme and saw that it was very inconsistent as described. I'll attach some screenshots.
            Hide
            Anthony Borrow added a comment -

            I am updating the affected version. While this goes back to previous versions, it is currently an issue in 1.9.4; however, any fix should be considered to be backported to other branches. Peace - Anthony

            Show
            Anthony Borrow added a comment - I am updating the affected version. While this goes back to previous versions, it is currently an issue in 1.9.4; however, any fix should be considered to be backported to other branches. Peace - Anthony
            Hide
            Dan Marsden added a comment -

            is this still a problem with the new 2.0 theme stuff?

            Show
            Dan Marsden added a comment - is this still a problem with the new 2.0 theme stuff?
            Hide
            Marina Glancy added a comment -

            We have detected that this issue has been inactive for over two years and also did not collect many votes. It is possible that it has been already implemented in a more recent version of Moodle, or it is not highly demanded. There are unlimited number of ways Moodle functinality can be expanded and improved but we would like to concentrate on the features that will benefit majority of users, and which can not be implemented as plugins. If you have a suggestion for improving Moodle core, and there is no open issue for it in the tracker, please start a new forum discussion to see how many other users agree with you, and then create a new issue providing as many details as possible.

            ==BLK2YIMP20141121==

            Show
            Marina Glancy added a comment - We have detected that this issue has been inactive for over two years and also did not collect many votes. It is possible that it has been already implemented in a more recent version of Moodle, or it is not highly demanded. There are unlimited number of ways Moodle functinality can be expanded and improved but we would like to concentrate on the features that will benefit majority of users, and which can not be implemented as plugins. If you have a suggestion for improving Moodle core, and there is no open issue for it in the tracker, please start a new forum discussion to see how many other users agree with you, and then create a new issue providing as many details as possible. ==BLK2YIMP20141121==

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: