Details

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

      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".

        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