-
Notifications
You must be signed in to change notification settings - Fork 173
Compat: remove parenthesis around No; add title attribute #230
Conversation
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.
👍 Looks good. There are some changes that we could make at the same time, or in new PRs.
macros/Compat.ejs
Outdated
@@ -103,10 +103,12 @@ function getVersionString(versionInfo) { | |||
style="color: rgb(255, 153, 0);">${localize(compatStrings, 'supportsShort_unknown')}</span>`; | |||
break; | |||
case true: | |||
return `<span style="color: #888">(${localize(compatStrings, 'supportsShort_yes')})</span>`; | |||
return `<span title="${localize(compatStrings, 'supportsShort_yes_title')}" | |||
style="color: #888">(${localize(compatStrings, 'supportsShort_yes')})</span>`; |
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.
@stephaniehobson:
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.
Should this be "(Yes)" in English and "«Да»" in Russian? In other words, should we move the parentheses inside the translated string?
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.
Updated!
break; | ||
case false: | ||
return `<span style="color: #f00">(${localize(compatStrings, 'supportsShort_no')})</span>`; | ||
return `<span title="${localize(compatStrings, 'supportsLong_no')}" | ||
style="color: #f00">${localize(compatStrings, 'supportsShort_no')}</span>`; |
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.
@stephaniehobson :
Hopefully these inline styles will go away when we switch to the new compat tables :) |
* mdn/kumascript#213 - CommunityBox, DiscussionList: Discourse * mdn/kumascript#222 - SpecName, Spec2: GamepadExtensions * mdn/kumascript#223 - Add folder for no macros test * mdn/kumascript#224 - AddonSidebar: Submit an add-on link * mdn/kumascript#225 - Compat: Alpha-sort browsers * mdn/kumascript#226 - make lint and linting changes * mdn/kumascript#228 - CompatNo: Add Dutch translation * mdn/kumascript#230 - Compat: Remove parens around No, title * mdn/kumascript#231 - Glossary: Add Ukranian support * mdn/kumascript#232 - AddonSidebar: De-emphasise legacy addons
"No" is currently rendered as "(No)". The parenthesis only make sense for "(Yes)" and we never had them for "No". Adding title tooltips to further clarify the meanings as well.