Moodle
  1. Moodle
  2. MDL-27064

Blogs: Different titles for different contexts

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical 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
          Jason Fowler added a comment -

          Up for peer review

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

          rebased after the weeklies

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

          Looks good. Thanks for the fix Jason.

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

          will cherry pick these across first thing tomorrow, thanks Adrian

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

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

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

          These strings are capitalised as they are link titles

          Show
          Jason Fowler added a comment - These strings are capitalised as they are link titles
          Hide
          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
          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
          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
          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
          Jason Fowler added a comment -

          all updated and ready to role

          Show
          Jason Fowler added a comment - all updated and ready to role
          Hide
          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
          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
          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
          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
          Sam Hemelryk added a comment -

          Thanks Jason, this has been integrated now

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

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

          Show
          Petr Skoda added a comment - Hmm, technically there are no course blogs anymore, maybe "Course related blog posts"?
          Hide
          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
          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
          Rossiani Wijaya added a comment -

          This looks good.

          Test passed.

          Show
          Rossiani Wijaya added a comment - This looks good. Test passed.
          Hide
          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
          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: