Move noscript styles to their own stylesheet

This way we can use SCSS syntax here, like we do everywhere else.
2024-04-12 15:26:25 +02:00
parent ce1ee861f1
commit 63eacf4579
5 changed files with 9 additions and 8 deletions
--- a/app/assets/config/manifest.js
+++ b/app/assets/config/manifest.js
@@ -10,6 +10,7 @@
 //= link stat_graphs.js
 //= link dashboard_graphs.js
 //= link dashboard_poster.css
+//= link noscript.css
 //= link print.css
 //= link pdf_fonts.css
 //= link_tree ../../../vendor/assets/images
--- a/app/assets/stylesheets/noscript.scss
+++ b/app/assets/stylesheets/noscript.scss
@@ -0,0 +1,5 @@
+.admin {
+  .tabs-content > .tabs-panel {
+    display: block;
+  }
+}
--- a/app/views/layouts/_common_head.html.erb
+++ b/app/views/layouts/_common_head.html.erb
@@ -11,3 +11,6 @@
 <%= javascript_include_tag "application", "data-turbolinks-track" => "reload" %>
 <%= csrf_meta_tags %>
 <%= favicon_link_tag image_path_for("favicon.ico") %>
+<noscript>
+  <%= stylesheet_link_tag "noscript" %>
+</noscript>
--- a/app/views/layouts/_no_script.html.erb
+++ b/app/views/layouts/_no_script.html.erb
@@ -1,7 +0,0 @@
-<noscript>
-  <style type="text/css">
-    .admin .tabs-content > .tabs-panel {
-      display: block;
-    }
-  </style>
-</noscript>
--- a/app/views/layouts/admin.html.erb
+++ b/app/views/layouts/admin.html.erb
@@ -2,7 +2,6 @@
 <html <%= common_html_attributes %>>
  <head>
    <%= render "layouts/common_head", default_title: "Admin" %>
-    <%= render "layouts/no_script" %>
    <%= content_for :head %>
  </head>