Convert remaining heading titles to use h1

This makes them consistent with all the other pages that use h1
in the `content_for :heading` sections

diff --git a/app/views/diary_entries/no_such_entry.html.erb b/app/views/diary_entries/no_such_entry.html.erb
index db8bc6f0940e56d96646f895e7f2ca00454267ec..da7d18524a90163fc3036d586e7b7a5abe713a4d 100644 (file)
--- a/app/views/diary_entries/no_such_entry.html.erb
+++ b/app/views/diary_entries/no_such_entry.html.erb
@@ -1,5 +1,5 @@
 <% content_for :heading do %>
-  <h2><%= t ".heading", :id => h(params[:id]) %></h2>
+  <h1><%= t ".heading", :id => h(params[:id]) %></h1>
 <% end %>
 
 <p><%= t ".body", :id => h(params[:id]) %></p>

diff --git a/app/views/diary_entries/show.html.erb b/app/views/diary_entries/show.html.erb
index 23b2dc783dd6ce31ac5299fd5e0a1f5b775e88ec..ccd647d3c6a3114c9ebe93707b2d9b6289a851d2 100644 (file)
--- a/app/views/diary_entries/show.html.erb
+++ b/app/views/diary_entries/show.html.erb
@@ -4,7 +4,7 @@
       <%= user_image @entry.user %>
     </div>
     <div class="col">
-      <h2><%= link_to t(".user_title", :user => @entry.user.display_name), :action => :index %></h2>
+      <h1><%= link_to t(".user_title", :user => @entry.user.display_name), :action => :index %></h1>
       <p><%= rss_link_to :action => :rss, :display_name => @entry.user.display_name %></p>
     </div>
   </div>

diff --git a/app/views/issues/show.html.erb b/app/views/issues/show.html.erb
index 5a457cea78d9146886fd186d72d4516d2cef03c1..ba2738b745fc042ac28d9c53fe5a6c7e4815dcd7 100644 (file)
--- a/app/views/issues/show.html.erb
+++ b/app/views/issues/show.html.erb
@@ -1,5 +1,5 @@
 <% content_for :heading do %>
-<h2><%= t ".title", :status => @issue.status.humanize, :issue_id => @issue.id %></h2>
+<h1><%= t ".title", :status => @issue.status.humanize, :issue_id => @issue.id %></h1>
 <p><%= @issue.reportable.model_name.human %> : <%= link_to reportable_title(@issue.reportable), reportable_url(@issue.reportable) %></p>
 <p class="text-muted">
   <small>

diff --git a/app/views/messages/new.html.erb b/app/views/messages/new.html.erb
index 663456ec7671eab40e7beb83f46cf2f05256b3f4..7c91722ff1b37045457ea689f3875d90d128956f 100644 (file)
--- a/app/views/messages/new.html.erb
+++ b/app/views/messages/new.html.erb
@@ -1,5 +1,5 @@
 <% content_for :heading do %>
-  <h2><%= t(".send_message_to_html", :name => link_to(@message.recipient.display_name, user_path(@message.recipient))) %></h2>
+  <h1><%= t(".send_message_to_html", :name => link_to(@message.recipient.display_name, user_path(@message.recipient))) %></h1>
 <% end %>
 
 <%= bootstrap_form_for @message do |f| %>

diff --git a/app/views/messages/show.html.erb b/app/views/messages/show.html.erb
index 74c61669fb21f2799ca6cdc1485b3ed2eaadf3ed..9ade68121aab29d9e7bac8f9bdcde705c87ac81c 100644 (file)
--- a/app/views/messages/show.html.erb
+++ b/app/views/messages/show.html.erb
@@ -1,5 +1,5 @@
 <% content_for :heading do %>
-  <h2><%= @message.title %></h2>
+  <h1><%= @message.title %></h1>
 <% end %>
 
 <% if current_user == @message.recipient %>

diff --git a/app/views/notes/index.html.erb b/app/views/notes/index.html.erb
index b5af25321df4cbcf8b1772ccdc078fd033de1775..a627e1c545b03f1ef773912115397a36b2439889 100644 (file)
--- a/app/views/notes/index.html.erb
+++ b/app/views/notes/index.html.erb
@@ -1,5 +1,5 @@
 <% content_for :heading do %>
-  <h2><%= t ".heading", :user => @user.display_name %></h2>
+  <h1><%= t ".heading", :user => @user.display_name %></h1>
   <p><%= t ".subheading_html", :user => link_to(@user.display_name, user_path(@user)) %></p>
 <% end %>
 

diff --git a/app/views/traces/edit.html.erb b/app/views/traces/edit.html.erb
index 13829727c9378b6443daf46af126ae82fc578cf3..7365295bd1176c96317c9cdf9aa1e0195e1ea454 100644 (file)
--- a/app/views/traces/edit.html.erb
+++ b/app/views/traces/edit.html.erb
@@ -1,5 +1,5 @@
 <% content_for :heading do %>
-  <h2><%= t ".heading", :name => @trace.name %></h2>
+  <h1><%= t ".heading", :name => @trace.name %></h1>
 <% end %>
 
 <img src="<%= url_for :controller => "traces", :action => "picture", :id => @trace.id, :display_name => @trace.user.display_name %>">

diff --git a/app/views/traces/offline.html.erb b/app/views/traces/offline.html.erb
index 8a26744fd2ac166fc0fec8c4d0599b6cca46a194..b3958d6a38450ec0a1ee387c3ceb957af3a02d7c 100644 (file)
--- a/app/views/traces/offline.html.erb
+++ b/app/views/traces/offline.html.erb
@@ -1,5 +1,5 @@
 <% content_for :heading do %>
-<h2><%= t ".heading" %></h2>
+  <h1><%= t ".heading" %></h1>
 <% end %>
 
 <p><%= t ".message" %></p>

diff --git a/app/views/traces/show.html.erb b/app/views/traces/show.html.erb
index 6df31760ab6e522dd2049db547e6925a57c36b58..e72c423d7c1e56918ae04860c76ce2665053daad 100644 (file)
--- a/app/views/traces/show.html.erb
+++ b/app/views/traces/show.html.erb
@@ -1,5 +1,5 @@
 <% content_for :heading do %>
-  <h2><%= t ".heading", :name => @trace.name %></h2>
+  <h1><%= t ".heading", :name => @trace.name %></h1>
 <% end %>
 
 <% if Settings.status != "gpx_offline" %>

diff --git a/test/controllers/diary_entries_controller_test.rb b/test/controllers/diary_entries_controller_test.rb
index 142363b60a21a8c5a38c17c14f281350a2df44f9..33e48365218e5ad70db6f722762d556277cb6fc6 100644 (file)
--- a/test/controllers/diary_entries_controller_test.rb
+++ b/test/controllers/diary_entries_controller_test.rb
@@ -273,7 +273,7 @@ class DiaryEntriesControllerTest < ActionDispatch::IntegrationTest
     get edit_diary_entry_path(:display_name => entry.user.display_name, :id => 9999)
     assert_response :not_found
     assert_select "div.content-heading", :count => 1 do
-      assert_select "h2", :text => "No entry with the id: 9999", :count => 1
+      assert_select "h1", :text => "No entry with the id: 9999", :count => 1
     end
 
     # Now pass the id, and check that you can edit it, when using the same
@@ -316,7 +316,7 @@ class DiaryEntriesControllerTest < ActionDispatch::IntegrationTest
     assert_template "show"
     assert_select "title", :text => /Users' Diaries | /, :count => 1
     assert_select "div.content-heading", :count => 1 do
-      assert_select "h2", :text => /#{entry.user.display_name}'s Diary/, :count => 1
+      assert_select "h1", :text => /#{entry.user.display_name}'s Diary/, :count => 1
     end
     assert_select "div#content", :count => 1 do
       assert_select "div.post_heading", :text => /#{new_title}/, :count => 1
@@ -336,7 +336,7 @@ class DiaryEntriesControllerTest < ActionDispatch::IntegrationTest
     assert_template "show"
     assert_select "title", :text => /Users' Diaries | /, :count => 1
     assert_select "div.content-heading", :count => 1 do
-      assert_select "h2", :text => /#{entry.user.display_name}'s Diary/, :count => 1
+      assert_select "h1", :text => /#{entry.user.display_name}'s Diary/, :count => 1
     end
     assert_select "div#content", :count => 1 do
       assert_select "div.post_heading", :text => /#{new_title}/, :count => 1
@@ -374,7 +374,7 @@ class DiaryEntriesControllerTest < ActionDispatch::IntegrationTest
     post comment_diary_entry_path(:display_name => entry.user.display_name, :id => 9999)
     assert_response :not_found
     assert_select "div.content-heading", :count => 1 do
-      assert_select "h2", :text => "No entry with the id: 9999", :count => 1
+      assert_select "h1", :text => "No entry with the id: 9999", :count => 1
     end
 
     # Now try an invalid comment with an empty body