Update footer links to current browse sections

Also update it so the list is in alphabetical order. As with the homepage the easiest way of doing this is to have two lists floated side by side.
alphagov · Oct 16, 2014 · 07ef2a1 · 07ef2a1
1 parent fd5d725
commit 07ef2a1
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 20 deletions.
diff --git a/app/assets/stylesheets/helpers/_footer.scss b/app/assets/stylesheets/helpers/_footer.scss
@@ -18,6 +18,7 @@
 
       h2 {
         margin: 0 15px;
+        padding: 10px 0 0;
 
         @include media(tablet) {
           padding: 0 0 20px;
@@ -40,26 +41,26 @@
     ul {
       @include core-16;
       list-style: none;
-
-      padding: 10px 0 0;
-      margin: 0 15px 30px;
+      padding: 0;
+      margin: 0;
 
       @include media(tablet){
-        padding: 20px 0 0;
-        margin: 0 15px;
-      }
-      @include ie-lte(7){
-        padding: 0; // for some reason IE6-7 gives the ul padding to each of the floated children
+        float: left;
+        margin: 15px 0 0 0;
+        width: 50%;
       }
+
 
       li {
         display: block;
         margin-bottom: 5px;
 
+        padding: 10px 0 0;
+        margin: 0 15px 5px;
+
         @include media(tablet){
-          float: left;
-          margin: 15px 0 0 0;
-          width: 50%;
+          padding: 20px 0 0;
+          margin: 0 15px;
         }
       }
     }

diff --git a/app/views/root/_base.html.erb b/app/views/root/_base.html.erb
@@ -44,19 +44,23 @@
       <h2>Services and information</h2>
 
       <ul>
-        <li><a href="/browse/driving">Driving and transport</a></li>
         <li><a href="/browse/benefits">Benefits</a></li>
-        <li><a href="/browse/business">Businesses and self-employed</a></li>
-        <li><a href="/browse/employing-people">Employing people</a></li>
-        <li><a href="/browse/abroad">Passports, travel and living abroad</a></li>
+        <li><a href="/browse/births-deaths-marriages">Births, deaths, marriages and care</a></li>
+        <li><a href="/browse/business">Business and self-employed</a></li>
+        <li><a href="/browse/citizenship">Citizenship and living in the UK</a></li>
+        <li><a href="/browse/justice">Crime, justice and the law</a></li>
+        <li><a href="/browse/disabilities">Disabled people</a></li>
+        <li><a href="/browse/driving">Driving and transport</a></li>
         <li><a href="/browse/education">Education and learning</a></li>
-        <li><a href="/browse/working">Working, jobs and pensions</a></li>
+      </ul>
+      <ul>
+        <li><a href="/browse/employing-people">Employing people</a></li>
+        <li><a href="/browse/environment-countryside">Environment and countryside</a></li>
         <li><a href="/browse/housing">Housing and local services</a></li>
-        <li><a href="/browse/justice">Crime, justice and the law</a></li>
         <li><a href="/browse/tax">Money and tax</a></li>
-        <li><a href="/browse/births-deaths-marriages">Births, deaths, marriages and care</a></li>
-        <li><a href="/browse/disabilities">Disabled people</a></li>
-        <li><a href="/browse/citizenship">Citizenship and life in the UK</a></li>
+        <li><a href="/browse/abroad">Passports, travel and living abroad</a></li>
+        <li><a href="/visas-immigration">Visas and immigration</a></li>
+        <li><a href="/browse/working">Working, jobs and pensions</a></li>
       </ul>
     </div>