feat(seo): shift h1 tag from logo to discussion title (#3724)

* feat(seo): shift h1 tag from logo to discussion title

Many times have we seen opponents of using h2 as the discussion title. Although
my own SEO knowledge is limited, I have seen the importance of structuring pages
according to the content you wish to prioritize. If we only take that into consideration
there is zero reason for the app-wide identical logo to take precedence over any
other heading.

This change makes the logo a standard (visually identical) element, and
makes the discussion hero title a h1.

* feat(seo): rewrite remainder of hero titles and subtitles

js/src/forum/components/TagHero.js

@@ -15,9 +15,9 @@ export default class TagHero extends Component {
       >
         <div className="container">
           <div className="containerNarrow">
-            <h2 className="Hero-title">
+            <h1 className="Hero-title">
               {tag.icon() && tagIcon(tag, {}, { useColor: false })} {tag.name()}
-            </h2>
+            </h1>
             <div className="Hero-subtitle">{tag.description()}</div>
           </div>
         </div>

views/frontend/content/tag.blade.php

@@ -1,7 +1,7 @@
 @inject('url', 'Flarum\Http\UrlGenerator')
 
 <div class="container">
-    <h2>{{ $tag->name }}</h2>
+    <h1>{{ $tag->name }}</h1>
     <p>{{ $tag->description }}</p>
 
     <ul>

views/frontend/content/tags.blade.php

@@ -1,7 +1,7 @@
 @inject('url', 'Flarum\Http\UrlGenerator')
 
 <div class="container">
-    <h2>{{ $translator->trans('flarum-tags.forum.index.tags_link') }}</h2>
+    <h1>{{ $translator->trans('flarum-tags.forum.index.tags_link') }}</h1>
 
     @foreach ([$primaryTags, $secondaryTags] as $category)
         <ul>