You are right.  I misunderstood the i8n team when they explained this -
thinking the translators comment could be anywhere above the string - as
long as no other string was in between.

I have now fixed all the Translators lines in this file.


On Thu, Mar 20, 2014 at 10:28 AM, David Baumgold
<notifications@github.com>wrote:

> In common/lib/xmodule/xmodule/tabs.py:
>
> > +    """
> > +    Abstract class for tabs that can be accessed by only users with staff access.
> > +    """
> > +    def can_display(self, course, settings, is_user_authenticated, is_user_staff):  # pylint: disable=unused-argument
> > +        return is_user_staff
> > +
> > +
> > +class CoursewareTab(CourseTab):
> > +    """
> > +    A tab containing the course content.
> > +    """
> > +
> > +    type = 'courseware'
> > +
> > +    def __init__(self, tab=None):  # pylint: disable=unused-argument
> > +        # Translators: 'Courseware' refers to the tab in the courseware that leads to the content of a course
>
> I don't think this will get picked up unless it's *directly* above the
> line that has the _('Courseware') in it.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-platform/pull/2910/files#r10796744>
> .
>



-- 

edX | nasthagiri@edx.org <andya@edx.org>

141 Portland Street, 9th floor

Cambridge, MA 02139
http://www.edx.org <http://www.edxonline.org/>

[image:
http://www.e-learn.nl/media/blogs/e-learn/edX_Logo_Col_RGB_FINAL.jpg?mtime=1336074566]