Adds tool tip for timeline toggle input field #1779
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adds a tooltip displaying what the timeline is good for, uses I18n locales.
|
Suggestions on the tooltip message will be appreciated. 😄 |
vzhz
reviewed
Mar 10, 2018
config/locales/en.yml
Outdated
| @@ -538,6 +538,7 @@ en: | |||
| timeline: Assignment Details | |||
| timeline_enabled: Timeline enabled | |||
| timeline_link: Timeline | |||
| timeline_tooltip: Timeline help you to keep track of your course by breaking the course duration in weeks which have proper agendas and tasks associated with them. | |||
There was a problem hiding this comment.
The only change I would make is a grammatical one: I would say "...into weeks with agendas and tasks associated with them." It's tighter as a whole and "into" is more clear because, for example, you can break something into pieces but don't break something in pieces. Just a suggestion! @psinghal20
ragesoss
reviewed
Mar 12, 2018
config/locales/en.yml
Outdated
| @@ -538,6 +538,7 @@ en: | |||
| timeline: Assignment Details | |||
| timeline_enabled: Timeline enabled | |||
| timeline_link: Timeline | |||
| timeline_tooltip: Timeline help you to keep track of your course by breaking the course duration into weeks which have proper agendas and tasks associated with them. | |||
There was a problem hiding this comment.
"Enabling the Timeline will let you specify week-by-week activities and assign training modules. It is useful for Education Program courses and other long, highly structured programs."
|
@ragesoss Updated the tooltip message. Thanks 😄 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1765.
Adds a tooltip displaying a message about what the timeline is good for, uses I18n locales.
Screenshot: