Factor out comment code from layouts into partials
authorTom Hughes <tom@compton.nu>
Sun, 3 Apr 2011 09:27:08 +0000 (10:27 +0100)
committerTom Hughes <tom@compton.nu>
Sun, 3 Apr 2011 12:46:17 +0000 (13:46 +0100)
app/views/layouts/_flash.html.erb [new file with mode: 0644]
app/views/layouts/_head.html.erb [new file with mode: 0644]
app/views/layouts/site.html.erb
app/views/layouts/slim.html.erb

diff --git a/app/views/layouts/_flash.html.erb b/app/views/layouts/_flash.html.erb
new file mode 100644 (file)
index 0000000..da559d5
--- /dev/null
@@ -0,0 +1,11 @@
+<% if flash[:error] %>
+  <div id="error"><%= flash[:error] %></div>
+<% end %>
+
+<% if flash[:warning] %>
+  <div id="warning"><%= flash[:warning] %></div>
+<% end %>
+
+<% if flash[:notice] %>
+  <div id="notice"><%= flash[:notice] %></div>
+<% end %>
diff --git a/app/views/layouts/_head.html.erb b/app/views/layouts/_head.html.erb
new file mode 100644 (file)
index 0000000..517e021
--- /dev/null
@@ -0,0 +1,18 @@
+<head>
+  <meta name="viewport" content="width=device-width, minimum-scale=1.0, maximum-scale=1.0"/>
+  <%= javascript_strings %>
+  <%= javascript_include_tag 'prototype' %>
+  <%= javascript_include_tag 'site' %>
+  <%= javascript_include_tag 'menu' %>
+  <!--[if lt IE 7]><%= javascript_include_tag 'pngfix' %><![endif]--> <!-- thanks, microsoft! -->
+  <%= stylesheet_link_tag 'common' %>
+  <!--[if IE]><%= stylesheet_link_tag 'large', :media => "screen" %><![endif]--> <!-- IE is totally broken with CSS media queries -->
+  <%= stylesheet_link_tag 'small', :media => "only screen and (max-width: 481px)" %>
+  <%= stylesheet_link_tag 'large', :media => "screen and (min-width: 482px)" %> 
+  <%= stylesheet_link_tag 'print', :media => "print" %>
+  <%= tag("link", { :rel => "search", :type => "application/opensearchdescription+xml", :title => "OpenStreetMap Search", :href => "/opensearch/osm.xml" }) %>
+  <%= tag("meta", { :name => "description", :content => "OpenStreetMap is the free wiki world map." }) %>
+  <%= style_rules %>
+  <%= yield :head %>
+  <title><%= t 'layouts.project_name.title' %><%= ' | '+ h(@title) if @title %></title>
+</head>
index b3f614257720399cf0dc1a220d2fe862acbe253c..d04a2877fcaa0318ad2921ebb7981b20660db35c 100644 (file)
@@ -1,34 +1,9 @@
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="<%= I18n.locale %>" lang="<%= I18n.locale %>" dir="<%= t'html.dir' %>">
-  <head>
-    <meta name="viewport" content="width=device-width, minimum-scale=1.0, maximum-scale=1.0"/>
-    <%= javascript_strings %>
-    <%= javascript_include_tag 'prototype' %>
-    <%= javascript_include_tag 'site' %>
-    <%= javascript_include_tag 'menu' %>
-    <!--[if lt IE 7]><%= javascript_include_tag 'pngfix' %><![endif]--> <!-- thanks, microsoft! -->
-    <%= stylesheet_link_tag 'common' %>
-    <!--[if IE]><%= stylesheet_link_tag 'large', :media => "screen" %><![endif]--> <!-- IE is totally broken with CSS media queries -->
-    <%= stylesheet_link_tag 'small', :media => "only screen and (max-width: 481px)" %>
-    <%= stylesheet_link_tag 'large', :media => "screen and (min-width: 482px)" %> 
-    <%= stylesheet_link_tag 'print', :media => "print" %>
-    <%= tag("link", { :rel => "search", :type => "application/opensearchdescription+xml", :title => "OpenStreetMap Search", :href => "/opensearch/osm.xml" }) %>
-    <%= tag("meta", { :name => "description", :content => "OpenStreetMap is the free wiki world map." }) %>
-    <%= style_rules %>
-    <%= yield :head %>
-    <title><%= t 'layouts.project_name.title' %><%= ' | '+ h(@title) if @title %></title>
-  </head>
+  <%= render :partial => "layouts/head" %>
   <body>
     <div id="content">
-      <% if flash[:error] %>
-        <div id="error"><%= flash[:error] %></div>
-      <% end %>
-      <% if flash[:warning] %>
-        <div id="warning"><%= flash[:warning] %></div>
-      <% end %>
-      <% if flash[:notice] %>
-        <div id="notice"><%= flash[:notice] %></div>
-      <% end %>
+      <%= render :partial => "layouts/flash", :locals => { :flash => flash } %>
 
       <%= yield %>
     </div>
index a97e526b1ef7f6012abd2628516eb02326107be8..862f55d874f0424af47049f1fe202ac47c7b9cfa 100644 (file)
@@ -1,36 +1,11 @@
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="<%= I18n.locale %>" lang="<%= I18n.locale %>" dir="<%= t'html.dir' %>">
-  <head>
-    <meta name="viewport" content="width=device-width, minimum-scale=1.0, maximum-scale=1.0"/>
-    <%= javascript_strings %>
-    <%= javascript_include_tag 'prototype' %>
-    <%= javascript_include_tag 'site' %>
-    <%= javascript_include_tag 'menu' %>
-    <!--[if lt IE 7]><%= javascript_include_tag 'pngfix' %><![endif]--> <!-- thanks, microsoft! -->
-    <%= stylesheet_link_tag 'common' %>
-    <!--[if IE]><%= stylesheet_link_tag 'large', :media => "screen" %><![endif]--> <!-- IE is totally broken with CSS media queries -->
-    <%= stylesheet_link_tag 'small', :media => "only screen and (max-width: 481px)" %>
-    <%= stylesheet_link_tag 'large', :media => "screen and (min-width: 482px)" %> 
-    <%= stylesheet_link_tag 'print', :media => "print" %>
-    <%= tag("link", { :rel => "search", :type => "application/opensearchdescription+xml", :title => "OpenStreetMap Search", :href => "/opensearch/osm.xml" }) %>
-    <%= tag("meta", { :name => "description", :content => "OpenStreetMap is the free wiki world map." }) %>
-    <%= style_rules %>
-    <%= yield :head %>
-    <title><%= t 'layouts.project_name.title' %><%= ' | '+ h(@title) if @title %></title>
-  </head>
+  <%= render :partial => "layouts/head" %>
   <body class="slim">
     <div id="slim_container">
       <div id="slim_container_content">
         <div id="slim_content">
-          <% if flash[:error] %>
-            <div id="error"><%= flash[:error] %></div>
-          <% end %>
-          <% if flash[:warning] %>
-            <div id="warning"><%= flash[:warning] %></div>
-          <% end %>
-          <% if flash[:notice] %>
-            <div id="notice"><%= flash[:notice] %></div>
-          <% end %>
+          <%= render :partial => "layouts/flash", :locals => { :flash => flash } %>
 
           <%= yield %>
         </div>
@@ -40,7 +15,5 @@
         </div>
       </div>
     </div>
-
-
   </body>
 </html>