Closed RubenVerborgh closed 8 years ago
+1
Awesome, thanks :smile:
This change is a mistake based on a common misunderstanding about HTML and it’s disappointing to see it getting enshrined in RASH.
@sideshowbarker What “common misunderstanding” do you mean? Please clarify why <i>
and <b>
would be more appropriate for emphasis and strong emphasis than <em>
and <strong>
(and don't forget to back that up with references to the HTML specification).
@RubenVerborgh I’ll pass.
RASH uses the wrong elements for emphasis:
<i>
element to “[represent] a span of text in an alternate voice or mood, or otherwise offset from the normal prose in a manner indicating a different quality of text, such as a taxonomic designation, a technical term, an idiomatic phrase from another language, transliteration, a thought, or a ship name in Western texts.”<b>
element to “[represent] a span of text to which attention is being drawn for utilitarian purposes without conveying any extra importance and with no implication of an alternate voice or mood, such as key words in a document abstract, product names in a review, actionable words in interactive text-driven software, or an article lede.”This pull request introduces the correct elements with the semantics intended by RASH, which is
<em>
for emphasis and<strong>
for strong emphasis.With this pull request RASH becomes compatible with HTML5 with regard to emphasis.