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:
    • Rank:
      16702

      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

        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 Škoda added a comment -

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

        Show
        Petr Škoda 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 Škoda added a comment -

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

        Show
        Petr Škoda 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: