Auto merge of #2101 - kzys:noscript, r=jtgeibel

bors · bors · commit 33dfe9931fe3 · 2020-01-09T01:21:50.000Z
Tweak &lt;noscript&gt; text and placement

After having FastBoot, JavaScript would not be "required" but still
necessary for full functionality of crates.io.
diff --git a/app/index.html b/app/index.html
@@ -23,14 +23,13 @@
     <link rel="search" href="/opensearch.xml" type="application/opensearchdescription+xml" title="Cargo">
   </head>
   <body>
-    {{content-for 'body'}}
     <noscript>
-        <div id="main">
-            <div class='noscript'>
-                This site requires JavaScript to be enabled.
-            </div>
-        </div>
+      <div id='noscript'>
+        For full functionality of this site it is necessary to enable JavaScript.
+      </div>
     </noscript>
+    
+    {{content-for 'body'}}
 
     <script src="{{rootURL}}assets/vendor.js"></script>
     <script src="{{rootURL}}assets/cargo.js"></script>
diff --git a/app/styles/app.scss b/app/styles/app.scss
@@ -40,6 +40,11 @@ body > div {
     }
 }
 
+#noscript {
+  color: white;
+  padding: 10px 0;
+}
+
 #main {
   display: flex;
   flex-direction: column;

Original file line number	Diff line number	Diff line change
`@@ -40,6 +40,11 @@ body > div {`
`40`	`40`	`}`
`41`	`41`	`}`
`42`	`42`
	`43`	`+#noscript {`
	`44`	`+ color: white;`
	`45`	`+ padding: 10px 0;`
	`46`	`+}`
	`47`	`+`
`43`	`48`	`#main {`
`44`	`49`	`display: flex;`
`45`	`50`	`flex-direction: column;`