Merge pull request #882 from sul-dlss/remove-unnecessary-code

Remove unnecessary code
sul-dlss · Dec 7, 2023 · 8337b2e · 8337b2e
2 parents be53e6a + 685f67a
commit 8337b2e
Show file tree

Hide file tree

Showing 14 changed files with 6 additions and 312 deletions.
diff --git a/app/assets/stylesheets/application.sass.scss b/app/assets/stylesheets/application.sass.scss
@@ -1,6 +1,5 @@
 @import 'deja-vu-font';
 @import 'sul-variables';
-@import 'searchworks-mixins';
 
 @import 'bootstrap-variables';
 @import 'bootstrap/scss/bootstrap';
@@ -17,14 +16,10 @@
 @import 'modules/record-view';
 @import 'modules/record-metadata-section';
 @import 'modules/search-bar';
-@import 'modules/skip-to-nav';
 @import 'modules/sul-footer';
 @import 'modules/sul-icons';
-@import 'modules/sul-toolbar';
 @import 'modules/top-navbar';
 
-@import 'print/styles';
-
 .record-metadata {
   margin-bottom: 2em;
 }
diff --git a/app/assets/stylesheets/base.scss b/app/assets/stylesheets/base.scss
@@ -1,36 +1,13 @@
-a {
-  &.disabled {
-    color: #ccc;
-    cursor: not-allowed;
-    text-decoration: none;
-  }
-}
-
 .su-underline {
 	text-decoration: none;
   border-bottom: 1px dotted $sul-link-color-border;
 }
 
 h1 {
   font-family: 'dejavu_sansextralight';
+  letter-spacing: -0.05em;
 }
 
 h3 {
   font-weight: 400;
-  line-height: 1.3em;
-  margin: 0 0 .4em 0;
-}
-
-dl {
-  margin-bottom: 15px;
-}
-
-label {
-  font-weight: normal;
-}
-
-.section {
-  p, li {
-    max-width: 45em;
-  }
 }
diff --git a/app/assets/stylesheets/bootstrap-variables.scss b/app/assets/stylesheets/bootstrap-variables.scss
@@ -23,10 +23,6 @@ $navbar-default-link-hover-color: $white;
 $navbar-default-link-active-color: $white;
 $navbar-default-toggle-hover-bg: transparent;
 
-// Pagination
-$pagination-active-bg: $sul-pagination-active-bg;
-$pagination-active-border: $sul-pagination-active-bg;
-
 // Tables
 $table-bg-accent: $color-beige-10;
 $table-border-color: #b6b1A9;

diff --git a/app/assets/stylesheets/layout.scss b/app/assets/stylesheets/layout.scss
@@ -11,7 +11,7 @@
 	border: none;
 }
 
-#topnav, #search-navbar, #main-container, #sul-footer, #masthead-container {
+#topnav, #search-navbar, #main-container, #sul-footer {
 	margin: 0 auto;
 	text-align: left;
 }
@@ -31,19 +31,19 @@ a#sul-logo {
 }
 
 @media (min-width: 768px) {
-  #topnav, #search-navbar, #main-container, #sul-footer, #masthead-container {
+  #topnav, #search-navbar, #main-container, #sul-footer {
     width: 750px;
   }
 }
 
 @media (min-width: 980px) {
-  #topnav, #search-navbar, #main-container, #sul-footer, #masthead-container {
+  #topnav, #search-navbar, #main-container, #sul-footer {
     width: 970px;
   }
 }
 
 @media (min-width: 1200px) {
-  #topnav, #search-navbar, #main-container, #sul-footer, #masthead-container {
+  #topnav, #search-navbar, #main-container, #sul-footer {
     width: 1170px;
   }
 }
@@ -53,12 +53,4 @@ a#sul-logo {
   #main-container, #search-navbar {
     padding: 0 15px;
   }
-
-  #search-navbar-container .search-form {
-    padding: 0;
-  }
-
-  #search-navbar .navbar-toggle {
-    margin-right: 0;
-  }
 }
diff --git a/app/assets/stylesheets/modules/feedback-form.scss b/app/assets/stylesheets/modules/feedback-form.scss
@@ -1,5 +1,5 @@
 #feedback-form {
-  border-bottom: 3px solid $sul-toolbar-border;
+  border-bottom: 3px solid $color-beige-40;
 }
 
 .feedback-alert {

diff --git a/app/assets/stylesheets/modules/search-bar.scss b/app/assets/stylesheets/modules/search-bar.scss
@@ -1,5 +1,4 @@
 #search-navbar-container {
-  padding: 0;
   background-color: $color-cardinal-red;
   box-shadow: 0 4px 8px -2px rgba(0, 0, 0, 0.2)
 }
diff --git a/app/assets/stylesheets/modules/skip-to-nav.scss b/app/assets/stylesheets/modules/skip-to-nav.scss
diff --git a/app/assets/stylesheets/modules/sul-toolbar.scss b/app/assets/stylesheets/modules/sul-toolbar.scss
diff --git a/app/assets/stylesheets/print/styles.scss b/app/assets/stylesheets/print/styles.scss
diff --git a/app/assets/stylesheets/searchworks-mixins.scss b/app/assets/stylesheets/searchworks-mixins.scss
diff --git a/app/assets/stylesheets/sul-styles/sul_footer.scss b/app/assets/stylesheets/sul-styles/sul_footer.scss
diff --git a/app/assets/stylesheets/sul-variables.scss b/app/assets/stylesheets/sul-variables.scss
@@ -3,36 +3,16 @@ $color-black: #000;
 $white: #ffffff;
 $color-cardinal-red: #8c1515;
 $color-blackish: #2e2d29;
-$gray-41-percent: #696969;
 $color-gray-60: #8a887d;
 $color-pantone-401: #b6b1a9;
 $color-beige-40: #d5d0c0;
-$color-beige-30: #e3dfd5;
-$color-beige-20: #e9e6df;
 $color-beige-10: #f2f1eb;
 $color-beige-05: #fbfbf9;
 
-$sul-footer-bg-color: $color-beige-10;
-$sul-footer-border-top-color: $color-beige-20;
-$footer-height: 200px;
-$sul-footer-shadow-color: rgba(0, 0, 0, 0.2);
-$sul-footer-shadow: 0 4px 8px -8px $sul-footer-shadow-color inset;
-
 $sul-footer-bg-color: $color-beige-10;
 $sul-topnavbar-link-color: $color-cardinal-red;
 $sul-link-color: #006CB8;
 $sul-link-color-border: $color-pantone-401;
-$sul-pagination-active-bg: $gray-41-percent;
 $sul-body-bg: $color-beige-05;
 $sul-text-color: $color-black;
-$sul-h2-border-color: $color-beige-30;
-$sul-skip-to-nav-bg: $color-beige-10;
-$sul-toolbar-bg: transparent;
-$sul-toolbar-border: $color-beige-40;
-$btn-sul-toolbar-bar-color: $color-cardinal-red;
-$btn-sul-toolbar-bar-hover-bg: $color-beige-30;
 $sul-shadow-color: rgba(0, 0, 0, .2);
-
-// Sizes
-$gallery-document-width: 176px;
-$gallery-document-height: 400px;
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
@@ -44,8 +44,6 @@
     <%= content_for(:head) %>
   </head>
   <body>
-    <%= render 'shared/skip_to_nav' %>
-
     <div id="su-wrap"> <!-- #su-wrap start -->
       <div id="su-content"> <!-- #su-content start -->
         <div id="outer-container" class="container-fluid">