New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CB-239: Reference review style guide when writing a review #169
Conversation
@@ -23,8 +23,10 @@ | |||
</div> | |||
<div class="form-group"> | |||
<small class="text-muted pull-left" style="font-style:italic;"> | |||
{{ _('You can use <a href="%(url)s" target="_blank">Markdown</a> syntax to apply custom formatting.', | |||
url='http://daringfireball.net/projects/markdown/syntax') }} | |||
{{ _('Make sure to keep the <a href="%(style_guide_url)s" target="_blank">style guide</a> in mind when writing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can write the statement such that it doesn't seem as a strict compulsion/constraint. Something like:
"You can read the style guide for writing good CritiqueBrainz reviews!"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amended commit.
{{ _('You can use <a href="%(url)s" target="_blank">Markdown</a> syntax to apply custom formatting.', | ||
url='http://daringfireball.net/projects/markdown/syntax') }} | ||
{{ _('You can read the <a href="%(style_guide_url)s" target="_blank">style guide</a> for writing good CritiqueBrainz reviews! | ||
Remember, you can use <a href="%(markdown_url)s" target="_blank">Markdown</a> syntax to apply custom formatting.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, I suggest we change Remember
to Also
or Furthermore
.
Also, the next sentence (about using markdown
) gets little out of focus. We can make use of a <br/>
bringing it to a new line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amended.
Jira ticket
Minor change - just added it before the "you can use markdown" message.