From 42de4fbd321a0161973522bd9a6df3597e4c2b2a Mon Sep 17 00:00:00 2001
From: Max Lehmann <max.lehmann@digital.cabinet-office.gov.uk>
Date: Wed, 21 Aug 2019 10:43:07 +0100
Subject: [PATCH] Remove outdated CSS

We no longer support IE6 and IE7 to the extent where specific stylesheets are
necessary and the IE8 stylesheet was doing nothing helpful. Removing these
improves the compile time and makes the dev environment more useable.
---
 app/assets/stylesheets/application-ie6.scss | 7 -------
 app/assets/stylesheets/application-ie7.scss | 6 ------
 app/assets/stylesheets/application-ie8.scss | 6 ------
 app/views/layouts/application.html.erb      | 5 +----
 config/initializers/assets.rb               | 4 ----
 5 files changed, 1 insertion(+), 27 deletions(-)
 delete mode 100644 app/assets/stylesheets/application-ie6.scss
 delete mode 100644 app/assets/stylesheets/application-ie7.scss
 delete mode 100644 app/assets/stylesheets/application-ie8.scss

diff --git a/app/assets/stylesheets/application-ie6.scss b/app/assets/stylesheets/application-ie6.scss
deleted file mode 100644
index 3d507bd28..000000000
--- a/app/assets/stylesheets/application-ie6.scss
+++ /dev/null
@@ -1,7 +0,0 @@
-// BASE STYLESHEET FOR IE 6 COMPILER
-
-$is-ie: true;
-$ie-version: 6;
-$mobile-ie6: false;
-
-@import "application";
diff --git a/app/assets/stylesheets/application-ie7.scss b/app/assets/stylesheets/application-ie7.scss
deleted file mode 100644
index 78a06dc2d..000000000
--- a/app/assets/stylesheets/application-ie7.scss
+++ /dev/null
@@ -1,6 +0,0 @@
-// BASE STYLESHEET FOR IE 7 COMPILER
-
-$is-ie: true;
-$ie-version: 7;
-
-@import "application";
diff --git a/app/assets/stylesheets/application-ie8.scss b/app/assets/stylesheets/application-ie8.scss
deleted file mode 100644
index 68ef7eeb0..000000000
--- a/app/assets/stylesheets/application-ie8.scss
+++ /dev/null
@@ -1,6 +0,0 @@
-// BASE STYLESHEET FOR IE 8 COMPILER
-
-$is-ie: true;
-$ie-version: 8;
-
-@import "application";
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index 282739496..3561bcef6 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -8,10 +8,7 @@
       <%= @content_item.page_title %> - GOV.UK
     <% end %>
   </title>
-  <!--[if gt IE 8]><!--><%= stylesheet_link_tag "application", integrity: true, crossorigin: 'anonymous' %><!--<![endif]-->
-  <!--[if IE 6]><%= stylesheet_link_tag "application-ie6" %><script>var ieVersion = 6;</script><![endif]-->
-  <!--[if IE 7]><%= stylesheet_link_tag "application-ie7" %><script>var ieVersion = 7;</script><![endif]-->
-  <!--[if IE 8]><%= stylesheet_link_tag "application-ie8" %><script>var ieVersion = 8;</script><![endif]-->
+  <%= stylesheet_link_tag "application", integrity: true, crossorigin: 'anonymous' %>
   <% if Rails.env.test? && params[:medium] == 'print' %>
     <%= stylesheet_link_tag "print.css", :media => "screen", integrity: true, crossorigin: 'anonymous' %>
   <% else %>
diff --git a/config/initializers/assets.rb b/config/initializers/assets.rb
index 1442f0fd9..fe56f2619 100644
--- a/config/initializers/assets.rb
+++ b/config/initializers/assets.rb
@@ -9,10 +9,6 @@
 # Precompile additional assets.
 # application.js, application.css, and all non-JS/CSS in app/assets folder are already added.
 Rails.application.config.assets.precompile += %w(
-  application-ie6.css
-  application-ie6.css
-  application-ie7.css
-  application-ie8.css
   print.css
   webchat.js
 )