Skip to content

Commit 1a92c01

Browse files
authored
Full website validation errors
Fixed validation errors in install/error/cli/benchmark pages. #335
1 parent c3999ad commit 1a92c01

File tree

5 files changed

+27
-23
lines changed

5 files changed

+27
-23
lines changed

website/public/benchmarks/criterion-json-to-html.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,8 @@ export function makeBenchmarkHTML(seriess) {
7171
<div class="x-axis-label">response time (lower is better)</div>
7272
<label
7373
class="log-scale javascript-only"
74-
title="checked: logarithmic X axis (emphasizing ratios).&#013;unchecked: linear X axis (emphasizing time)."
74+
title="checked: logarithmic X axis (emphasizing ratios).
75+
unchecked: linear X axis (emphasizing time)."
7576
><input type="checkbox" class="log-scale-select" /> log scale</label
7677
>
7778
</div>

website/public/cli/index.ejs.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<!DOCTYPE html>
22
<!-- Copyright (C) 2020 Matthew "strager" Glazar -->
33
<!-- See end of file for extended copyright information. -->
4-
<html>
4+
<html lang="en">
55
<head>
66
<%- await include("../common-head.ejs.html") %>
77
<script>

website/public/config/index.ejs.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<!DOCTYPE html>
22
<!-- Copyright (C) 2020 Matthew "strager" Glazar -->
33
<!-- See end of file for extended copyright information. -->
4-
<html>
4+
<html lang="en">
55
<head>
66
<%- await include("../common-head.ejs.html") %>
77
<script>

website/public/errors/index.ejs.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<!DOCTYPE html>
22
<!-- Copyright (C) 2020 Matthew "strager" Glazar -->
33
<!-- See end of file for extended copyright information. -->
4-
<html>
4+
<html lang="en">
55
<head>
66
<%- await include("../common-head.ejs.html") %>
77
<script>

website/public/install/index.ejs.html

Lines changed: 22 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<!DOCTYPE html>
22
<!-- Copyright (C) 2020 Matthew "strager" Glazar -->
33
<!-- See end of file for extended copyright information. -->
4-
<html>
4+
<html lang="en">
55
<head>
66
<%- await include("../common-head.ejs.html") %>
77
<title>quick-lint-js: installing</title>
@@ -86,7 +86,7 @@
8686
}
8787

8888
/* Stack items on top of each other if the screen isn't wide enough to
89-
arrange them horizontally. */
89+
arrange them horizontally. */
9090
@media only screen and (max-width: 34em) {
9191
.os-column-table {
9292
grid-template-areas: "linux" "macos" "windows";
@@ -97,7 +97,7 @@
9797
}
9898

9999
/* HACK(strager): Work around bug in EdgeHTML 13.10586 causing the Windows
100-
column of the operating systems column group to be super wide. */
100+
column of the operating systems column group to be super wide. */
101101
img.os-logo[src$="/windows.svg"] {
102102
width: 1.2em;
103103
}
@@ -127,7 +127,7 @@
127127
}
128128

129129
/* Add lots of spacing between different install procedures so people
130-
don't start reading the wrong instructions. */
130+
don't start reading the wrong instructions. */
131131
article {
132132
margin-top: 2rem;
133133
margin-bottom: 4rem;
@@ -138,6 +138,7 @@
138138

139139
.long-shell-command-line {
140140
/* 6 = "$ ".length + 4 spaces of indentation */
141+
display: inline-block;
141142
padding-left: 6ch;
142143
text-indent: -6ch;
143144
white-space: normal;
@@ -157,7 +158,10 @@ <h1><a href="..">quick-lint-js</a></h1>
157158

158159
<main>
159160
<h2>Install quick-lint-js</h2>
160-
<p>Latest version: 0.2.0 (beta) (released <time>April 5, 2021</time>)</p>
161+
<p>
162+
Latest version: 0.2.0 (beta) (released
163+
<time datetime="2021-04-05">April 5, 2021</time>)
164+
</p>
161165

162166
<figure class="comparison install-comparison">
163167
<table>
@@ -187,7 +191,7 @@ <h2>Install quick-lint-js</h2>
187191
class="install-logo"
188192
src="../gnome-terminal.svg"
189193
alt=""
190-
aria-labeled-by="install-artifact-cli"
194+
aria-labelledby="install-artifact-cli"
191195
/><br /><span id="install-artifact-cli">CLI</span>
192196
</th>
193197
<th>LSP server</th>
@@ -196,7 +200,7 @@ <h2>Install quick-lint-js</h2>
196200
class="install-logo"
197201
src="../vscode.png"
198202
alt=""
199-
aria-labeled-by="install-artifact-vscode"
203+
aria-labelledby="install-artifact-vscode"
200204
/><br /><span id="install-artifact-vscode"
201205
>VS&nbsp;Code<br />extension</span
202206
>
@@ -206,7 +210,7 @@ <h2>Install quick-lint-js</h2>
206210
class="install-logo"
207211
src="../vim.gif"
208212
alt=""
209-
aria-labeled-by="install-artifact-vim"
213+
aria-labelledby="install-artifact-vim"
210214
/><br /><span id="install-artifact-vim">Vim<br />plugin</span>
211215
</th>
212216
</tr>
@@ -219,7 +223,7 @@ <h2>Install quick-lint-js</h2>
219223
class="install-logo"
220224
src="../npm.svg"
221225
alt=""
222-
aria-labeled-by="install-method-npm"
226+
aria-labelledby="install-method-npm"
223227
/>&nbsp;<span id="install-method-npm">npm</span></a
224228
>
225229
</th>
@@ -286,7 +290,7 @@ <h2>Install quick-lint-js</h2>
286290
class="install-logo"
287291
src="../homebrew.svg"
288292
alt=""
289-
aria-labeled-by="install-method-homebrew"
293+
aria-labelledby="install-method-homebrew"
290294
/>&nbsp;<span id="install-method-homebrew">Homebrew</span></a
291295
>
292296
</th>
@@ -321,13 +325,13 @@ <h2>Install quick-lint-js</h2>
321325
class="install-logo"
322326
src="../debian.svg"
323327
alt=""
324-
aria-labeled-by="install-method-debian"
328+
aria-labelledby="install-method-debian"
325329
/>&nbsp;<span id="install-method-debian">Debian</span
326330
><br /><img
327331
class="install-logo"
328332
src="../ubuntu.svg"
329333
alt=""
330-
aria-labeled-by="install-method-ubuntu"
334+
aria-labelledby="install-method-ubuntu"
331335
/>&nbsp;<span id="install-method-ubuntu">Ubuntu</span></a
332336
>
333337
</th>
@@ -362,7 +366,7 @@ <h2>Install quick-lint-js</h2>
362366
class="install-logo"
363367
src="../nix.svg"
364368
alt=""
365-
aria-labeled-by="install-method-nix"
369+
aria-labelledby="install-method-nix"
366370
/>&nbsp;<span id="install-method-nix">Nix</span></a
367371
>
368372
</th>
@@ -402,7 +406,7 @@ <h2>Install quick-lint-js</h2>
402406
class="install-logo"
403407
src="../arch-linux.svg"
404408
alt=""
405-
aria-labeled-by="install-method-arch-linux"
409+
aria-labelledby="install-method-arch-linux"
406410
/>&nbsp;<span id="install-method-arch-linux"
407411
>Arch Linux</span
408412
></a
@@ -439,7 +443,7 @@ <h2>Install quick-lint-js</h2>
439443
class="install-logo"
440444
src="../vscode.png"
441445
alt=""
442-
aria-labeled-by="install-method-vscode"
446+
aria-labelledby="install-method-vscode"
443447
/>&nbsp;<span id="install-method-vscode"
444448
>VS&nbsp;Code<br />marketplace</span
445449
></a
@@ -488,7 +492,7 @@ <h2>Install quick-lint-js</h2>
488492
class="install-logo"
489493
src="../open-vsx.svg"
490494
alt=""
491-
aria-labeled-by="install-method-open-vsx"
495+
aria-labelledby="install-method-open-vsx"
492496
/>&nbsp;<span id="install-method-open-vsx"
493497
>Open&nbsp;VSX</span
494498
></a
@@ -990,7 +994,7 @@ <h2>
990994
quick-lint-js' tap to your Homebrew installation:
991995
</p>
992996
<blockquote>
993-
<pre><code><div class="long-shell-command-line"><kbd>brew tap quick-lint/quick-lint-js https://github.com/quick-lint/quick-lint-js.git</kbd></div></code></pre>
997+
<pre><code><span class="long-shell-command-line"><kbd>brew tap quick-lint/quick-lint-js https://github.com/quick-lint/quick-lint-js.git</kbd></span></code></pre>
994998
</blockquote>
995999

9961000
<p>
@@ -1034,7 +1038,7 @@ <h2>
10341038
<p>Open a terminal, and run the following commands:</p>
10351039
<blockquote>
10361040
<pre><code><kbd>curl https://c.quick-lint-js.com/quick-lint-js-release.key | sudo apt-key add -</kbd>
1037-
<div class="long-shell-command-line"><kbd>printf '\n# https://quick-lint-js.com/install/#debian\ndeb https://c.quick-lint-js.com/debian experimental main\n' | sudo tee -a /etc/apt/sources.list.d/quick-lint-js.list</kbd></div>
1041+
<span class="long-shell-command-line"><kbd>printf '\n# https://quick-lint-js.com/install/#debian\ndeb https://c.quick-lint-js.com/debian experimental main\n' | sudo tee -a /etc/apt/sources.list.d/quick-lint-js.list</kbd></span>
10381042
<kbd>sudo apt-get update</kbd>
10391043

10401044
# <img class="install-logo" src="../gnome-terminal.svg" alt="" /> CLI and LSP server
@@ -1540,7 +1544,6 @@ <h4 id="manual-vim-pathogen">Install with Pathogen</h4>
15401544
This option requires
15411545
<a href="https://github.com/tpope/vim-pathogen">Pathogen</a>.
15421546
</p>
1543-
<p />
15441547
<h5>
15451548
<img class="os-logo" src="../linux.svg" alt="" />&nbsp;GNU/Linux and
15461549
<img class="os-logo" src="../macos.svg" alt="" />&nbsp;macOS

0 commit comments

Comments
 (0)