From 33a6b79bd78f1903011367f6a5adf2f91d160f74 Mon Sep 17 00:00:00 2001
From: John Firebaugh <john.firebaugh@gmail.com>
Date: Tue, 16 Jul 2013 15:59:09 -0700
Subject: [PATCH] Add help page

---
 app/assets/stylesheets/common.css.scss | 43 +++++++++++++++++++++++++-
 app/controllers/site_controller.rb     |  3 ++
 app/views/layouts/site.html.erb        | 10 ++----
 app/views/site/help.html.erb           | 12 +++++++
 config/locales/en.yml                  | 31 +++++++++++++------
 config/routes.rb                       |  1 +
 6 files changed, 83 insertions(+), 17 deletions(-)
 create mode 100644 app/views/site/help.html.erb

diff --git a/app/assets/stylesheets/common.css.scss b/app/assets/stylesheets/common.css.scss
index 42467adb3..c9b7046f9 100644
--- a/app/assets/stylesheets/common.css.scss
+++ b/app/assets/stylesheets/common.css.scss
@@ -1108,7 +1108,8 @@ ul.results-list li { border-bottom: 1px solid #ccc; }
 .user-terms,
 .user-confirm,
 .site-copyright,
-.site-welcome {
+.site-welcome,
+.site-help {
   #content {
     max-width: 740px;
   }
@@ -2559,3 +2560,43 @@ a.button {
     border-radius: 4px;
   }
 }
+
+.site-help {
+  #content {
+    a {
+      display: inline-block;
+      width: 45%;
+      height: 200px;
+      float: left;
+      text-align: center;
+      color: #000;
+      margin: 10px;
+      padding: 10px;
+      border-radius: 10px;
+    }
+
+    h3 {
+      margin-top: 50px;
+    }
+
+    a:hover {
+      text-decoration: none;
+    }
+
+    .welcome {
+      background-color: #D6FFB9;
+    }
+
+    .learnosm {
+      background-color: #BAFCFF;
+    }
+
+    .help {
+      background-color: #FCFFB9;
+    }
+
+    .wiki {
+      background-color: #FFB9CC;
+    }
+  }
+}
diff --git a/app/controllers/site_controller.rb b/app/controllers/site_controller.rb
index 280f39fd8..4945aaa4f 100644
--- a/app/controllers/site_controller.rb
+++ b/app/controllers/site_controller.rb
@@ -71,6 +71,9 @@ class SiteController < ApplicationController
   def welcome
   end
 
+  def help
+  end
+
   def preview
     render :text => RichText.new(params[:format], params[:text]).to_html
   end
diff --git a/app/views/layouts/site.html.erb b/app/views/layouts/site.html.erb
index ed8a6cf18..461257207 100644
--- a/app/views/layouts/site.html.erb
+++ b/app/views/layouts/site.html.erb
@@ -39,13 +39,6 @@
       </div>
       <% end %>
       <ul id="left_menu" class="left_menu">
-        <li>
-          <h4><%= t'layouts.help' %></h4>
-          <ul>
-            <li><%= link_to(t('layouts.help_centre'), t('layouts.help_url'), :title => t('layouts.help_title')) %></li>
-            <li><%= link_to(t('layouts.documentation'), t('layouts.wiki_url'), :title => t('layouts.documentation_title')) %></li>
-          </ul>
-        </li>
         <li>
           <h4><%= t'layouts.community' %></h4>
           <ul>
@@ -84,6 +77,9 @@
       </div>
     </div>
     <div id='top-bar'>
+      <ul class='secondary-actions'>
+        <li><%= link_to t('layouts.help'), help_path %></li>
+      </ul>
       <% if @user and @user.id %>
         <%= render :partial => "layouts/user_menu" %>
       <% else %>
diff --git a/app/views/site/help.html.erb b/app/views/site/help.html.erb
new file mode 100644
index 000000000..e72aac190
--- /dev/null
+++ b/app/views/site/help.html.erb
@@ -0,0 +1,12 @@
+<% content_for :heading do %>
+  <h2><%= t "help_page.title" %></h2>
+<% end %>
+
+<p class='introduction'><%= t "help_page.introduction" %></p>
+
+<% ['welcome', 'learnosm', 'help', 'wiki'].each do |site| %>
+  <a class='<%= site %>' href='<%= t "help_page.#{site}.url" %>'>
+    <h3><%= t "help_page.#{site}.title" %></h3>
+    <p><%= t "help_page.#{site}.description" %></p>
+  </a>
+<% end %>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index a1938079f..302c3dbd3 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -1032,15 +1032,7 @@ en:
     osm_read_only: "The OpenStreetMap database is currently in read-only mode while essential database maintenance work is carried out."
     donate: "Support OpenStreetMap by %{link} to the Hardware Upgrade Fund."
     donate_link_text: donating
-    help: Help
-    help_centre: Help Centre
-    help_url: http://help.openstreetmap.org/
-    help_title: Help site for the project
-    wiki: Wiki
-    wiki_url: http://wiki.openstreetmap.org/
-    wiki_title: Wiki site for the project
-    documentation: Documentation
-    documentation_title: Documentation for the project
+    help: help
     copyright: "Copyright & License"
     community: Community
     community_blogs: "Community Blogs"
@@ -1208,6 +1200,27 @@ en:
         Just go to <a href='%{map_url}'>the map</a> and click the note icon:
         <span class='icon note'></span>. This will add a marker to the map, which you can move
         by dragging. Add your message, then click save, and other mappers will investigate.
+  help_page:
+    title: Getting Help
+    introduction: |
+      OpenStreetMap has several resources for learning about the project, asking and answering questions,
+      and collaboratively discussing and documenting mapping topics.
+    welcome:
+      url: http://www.openstreetmap.org/welcome
+      title: Welcome to OSM
+      description: A one-page introduction for new users
+    learnosm:
+      url: http://learnosm.org/
+      title: LearnOSM
+      description: Learn OpenStreetMap Step by Step
+    help:
+      url: https://help.openstreetmap.org/
+      title: help.openstreetmap.org
+      description: OSM's question-and-answer site
+    wiki:
+      url: http://wiki.openstreetmap.org/
+      title: wiki.openstreetmap.org
+      description: A wiki-based documentation site
   notifier:
     diary_comment_notification:
       subject: "[OpenStreetMap] %{user} commented on your diary entry"
diff --git a/config/routes.rb b/config/routes.rb
index 445c0353f..31658e4a9 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -126,6 +126,7 @@ OpenStreetMap::Application.routes.draw do
   match '/copyright/:copyright_locale' => 'site#copyright', :via => :get
   match '/copyright' => 'site#copyright', :via => :get
   match '/welcome' => 'site#welcome', :via => :get, :as => :welcome
+  match '/help' => 'site#help', :via => :get, :as => :help
   match '/history' => 'changeset#list', :via => :get
   match '/history/feed' => 'changeset#feed', :via => :get, :defaults => { :format => :atom }
   match '/export' => 'site#index', :export => true, :via => :get
-- 
2.43.2