Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-28566

Links in "user details" participant list appear inconsistently across themes

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.4, 2.1.1, 2.2
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: Administration, Themes
    • Labels:
    • Environment:
      Win7, XAMPP, FF or Chrome
    • Testing Instructions:
      Hide

      1. Log in as admin
      2. Navigate to Site Administration -> Appearance -> Theme -> Theme selector.
      3. Click the button labelled "Select theme" in the default row
      4. Choose either Standard, Afterburner or Anomaly
      5. Navigate to Navigation -> Site pages -> Participants
      6. On the right, next to User list, choose User details

      The links on the right (to Blog, Notes, etc.) should each appear on a separate line.

      Show
      1. Log in as admin 2. Navigate to Site Administration -> Appearance -> Theme -> Theme selector. 3. Click the button labelled "Select theme" in the default row 4. Choose either Standard, Afterburner or Anomaly 5. Navigate to Navigation -> Site pages -> Participants 6. On the right, next to User list, choose User details The links on the right (to Blog, Notes, etc.) should each appear on a separate line.
    • Workaround:
      Hide

      For the theme in use, add the included CSS snippet to core.css.

      Be sure to purge caches afterwards.

      Show
      For the theme in use, add the included CSS snippet to core.css. Be sure to purge caches afterwards.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull Master Branch:

      Description

      When using some themes, such as Standard, Afterburner and Anomaly, the list of user links for each participant as a single continuous line of text, whereas with other themes, such as Arialist, the list is formatted nicely with one link per line. This happens when viewing the participants list with user details shown.

      I found that some themes control the display as "block" and some do not. The following CSS should probably be present in all themes...

      core.css

      .userinfobox .links a {
          display: block;
      }

      Replication instructions:

      1. Log in as admin
      2. Navigate to Site Administration -> Appearance -> Theme -> Theme selector.
      3. Click the button labelled "Select theme" in the default row
      4. Choose either Standard, Afterburner or Anomaly
      5. Navigate to Navigation -> Site pages -> Participants
      6. On the right, next to User list, choose User details

      There are other ways to get to this view. It is used in courses.

        Gliffy Diagrams

          Attachments

          1. Afterburner.png
            Afterburner.png
            111 kB
          2. Anomaly.png
            Anomaly.png
            107 kB
          3. Arialist.png
            Arialist.png
            120 kB
          4. Standard.png
            Standard.png
            120 kB

            Issue Links

              Activity

              Hide
              dougiamas Martin Dougiamas added a comment -

              I heard a report from Jerome recently that all of the new themes are NOT inheriting properly from base and canvas. Could this be the problem? Surely it makes sense to have basic CSS like this in the base theme so that it's fixed in all themes.

              Show
              dougiamas Martin Dougiamas added a comment - I heard a report from Jerome recently that all of the new themes are NOT inheriting properly from base and canvas. Could this be the problem? Surely it makes sense to have basic CSS like this in the base theme so that it's fixed in all themes.
              Hide
              lazydaisy Mary Evans added a comment -

              That was easy!

              Show
              lazydaisy Mary Evans added a comment - That was easy!
              Hide
              salvetore Michael de Raadt added a comment -

              Thanks Mary.

              Show
              salvetore Michael de Raadt added a comment - Thanks Mary.
              Hide
              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
              samhemelryk Sam Hemelryk added a comment -

              Nearly forgot to mention - in future Mary please don't add comments referencing the MDL issue number into the changes.
              We can always get that information from tracker, git, or web interfaces like cvs.moodle.org, git.moodle.org etc - mostly they are just adding content that needs to be removed and will likely be outdated next time someone changes that line again or revises code.

              Cheers
              Sam

              Show
              samhemelryk Sam Hemelryk added a comment - Nearly forgot to mention - in future Mary please don't add comments referencing the MDL issue number into the changes. We can always get that information from tracker, git, or web interfaces like cvs.moodle.org, git.moodle.org etc - mostly they are just adding content that needs to be removed and will likely be outdated next time someone changes that line again or revises code. Cheers Sam
              Hide
              salvetore Michael de Raadt added a comment -

              Test results: works a treat.

              Show
              salvetore Michael de Raadt added a comment - Test results: works a treat.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              git repositories have been updated with your awesome changes, thanks! Closing.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - git repositories have been updated with your awesome changes, thanks! Closing.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    10/Oct/11