Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Component/s: moodle.org
    • Labels:

      Description

      There are a couple of confusing aspects to the moodle.org/stats page.

      • The moodle.org population graph is nice to see, but putting it in where it is changes the context of the document and makes the content that comes after it confusing. It should be either dropped from the page or shifted to the very end.
      • The country chart needs a title: "Top 10 countries by registrations".
      • The version pie graphs need a title: "Versions used".
      • The label above the left version pie chart is currently "Moodle registrations in the last two months" but should be "New Moodle registrations in the last two months".

        Gliffy Diagrams

          Activity

          Hide
          Helen Foster added a comment -

          Michael, thanks for your report. I agree with all your points.

          Assigning to moodle.com as unfortunately I am unable to edit the stats page.

          Show
          Helen Foster added a comment - Michael, thanks for your report. I agree with all your points. Assigning to moodle.com as unfortunately I am unable to edit the stats page.
          Hide
          Dan Poltawski added a comment -

          Another problem discovered when doing some of this - only half of the strings are translatable. See http://moodle.org/stats/?lang=fr for example

          Show
          Dan Poltawski added a comment - Another problem discovered when doing some of this - only half of the strings are translatable. See http://moodle.org/stats/?lang=fr for example
          Hide
          Dan Poltawski added a comment -

          I have fixed all these problems - just waiting for a moodle.org DB dump to test against.

          Show
          Dan Poltawski added a comment - I have fixed all these problems - just waiting for a moodle.org DB dump to test against.
          Hide
          Dan Poltawski added a comment -

          On the translating - the graphs will need to be static due to the way they are generated unfortunately so I reverted my change there.

          Show
          Dan Poltawski added a comment - On the translating - the graphs will need to be static due to the way they are generated unfortunately so I reverted my change there.
          Hide
          Dan Poltawski added a comment -

          These changes have now been deployed

          Show
          Dan Poltawski added a comment - These changes have now been deployed
          Hide
          Helen Foster added a comment -

          I notice all of Michael's suggestions have been implemented, apart from the last - the label reads 'Moodle registrations for the last two months', however I guess this is because it's part of the pie chart. I think the other suggestions were the most important and so I'm closing this issue. Thanks Dan

          Show
          Helen Foster added a comment - I notice all of Michael's suggestions have been implemented, apart from the last - the label reads 'Moodle registrations for the last two months', however I guess this is because it's part of the pie chart. I think the other suggestions were the most important and so I'm closing this issue. Thanks Dan
          Hide
          Dan Poltawski added a comment -

          Doh - I did that but got rid of it when reverting my language changes.

          Show
          Dan Poltawski added a comment - Doh - I did that but got rid of it when reverting my language changes.
          Hide
          Dan Poltawski added a comment -

          I have pushed the fix to the last issue - this will be deployed next time I get jordan to deploy some other changes.

          Show
          Dan Poltawski added a comment - I have pushed the fix to the last issue - this will be deployed next time I get jordan to deploy some other changes.
          Hide
          Michael de Raadt added a comment -

          Thanks for working on this, Dan.

          Show
          Michael de Raadt added a comment - Thanks for working on this, Dan.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development