From 44aeee5d8341337c00298bb51f069444af39a256 Mon Sep 17 00:00:00 2001 From: jdlrobson Date: Tue, 14 Dec 2021 09:40:37 -0800 Subject: [PATCH] Vector uses html-title-heading The H1 is available with the correct attributes in core now, so we can use that instead of our own markup. This also means Vector will benefit from updates to the heading in core such as main page special handling. Bug: T297728 Change-Id: I4d10370678fad6455d73994bf340891760543b71 --- includes/templates/skin-legacy.mustache | 2 +- includes/templates/skin.mustache | 4 ++-- stories/skin.stories.data.js | 1 + 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/includes/templates/skin-legacy.mustache b/includes/templates/skin-legacy.mustache index 3ac6492..0e998f3 100644 --- a/includes/templates/skin-legacy.mustache +++ b/includes/templates/skin-legacy.mustache @@ -30,7 +30,7 @@
{{{html-site-notice}}}
{{>Indicators}} -

{{{html-title}}}

+ {{{html-title-heading}}}
{{#is-article}}
{{msg-tagline}}
{{/is-article}}
{{{html-subtitle}}}
diff --git a/includes/templates/skin.mustache b/includes/templates/skin.mustache index e7e77a3..549824b 100644 --- a/includes/templates/skin.mustache +++ b/includes/templates/skin.mustache @@ -56,7 +56,7 @@ {{^is-language-in-content}} {{>Indicators}} -

{{{html-title}}}

+ {{{html-title-heading}}} {{/is-language-in-content}} {{#is-language-in-content}} @@ -64,7 +64,7 @@ {{#is-language-in-content-top}} {{#data-portlets.data-languages}}{{>Menu}}{{/data-portlets.data-languages}} {{/is-language-in-content-top}} -

{{{html-title}}}

+ {{{html-title-heading}}} {{>Indicators}} {{#is-article}}
{{msg-tagline}}
diff --git a/stories/skin.stories.data.js b/stories/skin.stories.data.js index 9a4d177..086c87f 100644 --- a/stories/skin.stories.data.js +++ b/stories/skin.stories.data.js @@ -88,6 +88,7 @@ const DATA_INDICATORS = [ { export const LEGACY_TEMPLATE_DATA = { 'html-title': 'Vector 2019', + 'html-title-heading': '

Vector 2019

', 'is-article': true, 'msg-tagline': 'From Wikipedia, the free encyclopedia', 'html-user-language-attributes': htmlUserLanguageAttributes,