Merge pull request #471 from alphagov/new-footer-links

Update footer links to current browse sections
alphagov · Oct 16, 2014 · 851de85 · 851de85
2 parents f45ad21 + 07ef2a1
commit 851de85
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>