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

Blogs: Different titles for different contexts

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0.2, 2.2.2
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Blog, Usability
    • Labels:

      Description

      In blocks is teh item 'Blog' used at several places.It links to different places with different functionalities.
      Site level and course level title for 'blog' should be recognizable as different functions: All of them are using the same language string 'blogs' from core_blogs.

      Please add two more language strings. My suggestion:
      'Site Blogs' at Site pages and
      'Course Blogs' at Navigation and
      'My blogs' at My profile.

      http://moodle.org/mod/forum/discuss.php?d=172388

        Gliffy Diagrams

          Activity

          Hide
          phalacee Jason Fowler added a comment -

          Up for peer review

          Show
          phalacee Jason Fowler added a comment - Up for peer review
          Hide
          phalacee Jason Fowler added a comment -

          rebased after the weeklies

          Show
          phalacee Jason Fowler added a comment - rebased after the weeklies
          Hide
          abgreeve Adrian Greeve added a comment -

          Looks good. Thanks for the fix Jason.

          Show
          abgreeve Adrian Greeve added a comment - Looks good. Thanks for the fix Jason.
          Hide
          phalacee Jason Fowler added a comment -

          will cherry pick these across first thing tomorrow, thanks Adrian

          Show
          phalacee Jason Fowler added a comment - will cherry pick these across first thing tomorrow, thanks Adrian
          Hide
          skodak Petr Skoda added a comment -

          to integrators: please review new strings, I guess only first word should be capitalised (ask David if necessary)

          Show
          skodak Petr Skoda added a comment - to integrators: please review new strings, I guess only first word should be capitalised (ask David if necessary)
          Hide
          phalacee Jason Fowler added a comment -

          These strings are capitalised as they are link titles

          Show
          phalacee Jason Fowler added a comment - These strings are capitalised as they are link titles
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Adding Helen for clarification and reopening. I think Petr is correct and camel case titles should not be used there.

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Adding Helen for clarification and reopening. I think Petr is correct and camel case titles should not be used there.
          Hide
          phalacee Jason Fowler added a comment -

          fair enough, I just looked at the other links around it and you're right, I'll make the change and re-push

          Show
          phalacee Jason Fowler added a comment - fair enough, I just looked at the other links around it and you're right, I'll make the change and re-push
          Hide
          phalacee Jason Fowler added a comment -

          all updated and ready to role

          Show
          phalacee Jason Fowler added a comment - all updated and ready to role
          Hide
          tsala Helen Foster added a comment -

          Thanks Jason, I can confirm that only the first word should be capitalised, as mentioned here: http://docs.moodle.org/dev/Coding_style#Language_strings

          Show
          tsala Helen Foster added a comment - Thanks Jason, I can confirm that only the first word should be capitalised, as mentioned here: http://docs.moodle.org/dev/Coding_style#Language_strings
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Jason, this has been integrated now

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Jason, this has been integrated now
          Hide
          skodak Petr Skoda added a comment -

          Hmm, technically there are no course blogs anymore, maybe "Course related blog posts"?

          Show
          skodak Petr Skoda added a comment - Hmm, technically there are no course blogs anymore, maybe "Course related blog posts"?
          Hide
          phalacee Jason Fowler added a comment -

          That's a little wordy to be going in the navigation, I think "Course blogs" is enough, it's the blogs of the users in that course ...

          Show
          phalacee Jason Fowler added a comment - That's a little wordy to be going in the navigation, I think "Course blogs" is enough, it's the blogs of the users in that course ...
          Hide
          rwijaya Rossiani Wijaya added a comment -

          This looks good.

          Test passed.

          Show
          rwijaya Rossiani Wijaya added a comment - This looks good. Test passed.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Congratulations are in order, you've made it, or at least your code has!
          It's now part of Moodle and both the git and cvs repositories have been updated.

          This issue is being marked as fixed and closed.

          Thank you.

          Show
          samhemelryk Sam Hemelryk added a comment - Congratulations are in order, you've made it, or at least your code has! It's now part of Moodle and both the git and cvs repositories have been updated. This issue is being marked as fixed and closed. Thank you.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/May/12