Move style to head to make routes page valid html5

2013-01-05 11:57:13 -02:00 · 2013-01-05 11:57:13 -02:00 · 60e809cffd
parent 82023532d8
commit 60e809cffd
3 changed files with 8 additions and 4 deletions
--- a/actionpack/lib/action_dispatch/middleware/templates/rescues/layout.erb
+++ b/actionpack/lib/action_dispatch/middleware/templates/rescues/layout.erb
@ -113,6 +113,8 @@
    a { color: #980905; }
    a:visited { color: #666; }
    a:hover { color: #C52F24; }
+
+    <%= yield :style %>
  </style>

  <script>
--- a/actionpack/lib/action_dispatch/middleware/templates/routes/_route_wrapper.html.erb
+++ b/actionpack/lib/action_dispatch/middleware/templates/routes/_route_wrapper.html.erb
@ -1,7 +1,7 @@
-<style type='text/css'>
-  #route_table td {padding: 0 30px;}
-  #route_table    {margin:  0 auto 0;}
-</style>
+<% content_for :style do %>
+  #route_table td { padding: 0 30px; }
+  #route_table    { margin:  0 auto 0; }
+<% end %>

 <table id='route_table' class='route_table'>
  <thead>
--- a/railties/lib/rails/templates/layouts/application.html.erb
+++ b/railties/lib/rails/templates/layouts/application.html.erb
@ -24,6 +24,8 @@
    a:hover { color: #fff; background-color:#000; }

    h2 { padding-left: 10px; }
+
+    <%= yield :style %>
  </style>
 </head>
 <body>