From cda7f1dc6c4694799f74004745e952b4f8fbca9c Mon Sep 17 00:00:00 2001 From: Oksana Slusarenro Date: Thu, 15 Sep 2016 10:24:54 +0300 Subject: [PATCH] Changed the Help and Field Label text. --- .../lib/xmodule/xmodule/conditional_module.py | 18 ++++++++++-------- .../xmodule/xmodule/tests/test_conditional.py | 2 +- 2 files changed, 11 insertions(+), 9 deletions(-) diff --git a/common/lib/xmodule/xmodule/conditional_module.py b/common/lib/xmodule/xmodule/conditional_module.py index a65abf0df6..4fb13e8519 100644 --- a/common/lib/xmodule/xmodule/conditional_module.py +++ b/common/lib/xmodule/xmodule/conditional_module.py @@ -39,15 +39,16 @@ class ConditionalFields(object): sources_list = ReferenceList( display_name=_("Source Components"), - help=_("The location IDs of the components whose attributes are used to determine whether a learner is shown " - "the content of this conditional module."), + help=_("The component location IDs of all source components that are used to determine whether a learner is " + "shown the content of this conditional module. Copy the component location ID of a component from its " + "Settings dialog in Studio."), scope=Scope.content ) conditional_attr = String( display_name=_("Conditional Attribute"), - help=_("The attribute from the course component used to determine whether a learner is shown " - "the content of this conditional module."), + help=_("The attribute of the source components that determines whether a learner is shown the content of this " + "conditional module."), scope=Scope.content, default='correct', values=lambda: [{'display_name': xml_attr, 'value': xml_attr} @@ -56,7 +57,7 @@ class ConditionalFields(object): conditional_value = String( display_name=_("Conditional Value"), - help=_("The value of the conditional attribute that must be true for a learner to be shown " + help=_("The value that the conditional attribute of the source components must match before a learner is shown " "the content of this conditional module."), scope=Scope.content, default='True' @@ -64,10 +65,11 @@ class ConditionalFields(object): conditional_message = String( display_name=_("Blocked Content Message"), - help=_("The message learners see when not all conditions are met for this block. " - "You can use the {link} variable to give learners a direct link to the required module."), + help=_("The message that is shown to learners when not all conditions are met to show the content of this " + "conditional module. Include {link} in the text of your message to give learners a direct link to " + "required units. For example, 'You must complete {link} before you can access this unit'."), scope=Scope.content, - default=_('{link} must be attempted before this will become visible.') + default=_('You must complete {link} before you can access this unit.') ) diff --git a/common/lib/xmodule/xmodule/tests/test_conditional.py b/common/lib/xmodule/xmodule/tests/test_conditional.py index e33623f350..8aae66b1ea 100644 --- a/common/lib/xmodule/xmodule/tests/test_conditional.py +++ b/common/lib/xmodule/xmodule/tests/test_conditional.py @@ -335,7 +335,7 @@ class ConditionalModuleXmlTest(unittest.TestCase): modules['cond_module'].definition_to_xml(Mock()) expected_xml_attributes = { 'attempted': 'true', - 'message': '{link} must be attempted before this will become visible.', + 'message': 'You must complete {link} before you can access this unit.', 'sources': '' } self.assertDictEqual(modules['cond_module'].xml_attributes, expected_xml_attributes)