1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
|
//! Tidy check to make sure light and dark themes are synchronized between
//! JS-controlled rustdoc.css and noscript.css
use std::path::Path;
use crate::diagnostics::{CheckId, DiagCtx, RunningCheck};
pub fn check(librustdoc_path: &Path, diag_ctx: DiagCtx) {
let mut check = diag_ctx.start_check(CheckId::new("rustdoc_css_themes").path(librustdoc_path));
let rustdoc_css = "html/static/css/rustdoc.css";
let noscript_css = "html/static/css/noscript.css";
let rustdoc_css_contents = std::fs::read_to_string(librustdoc_path.join(rustdoc_css))
.unwrap_or_else(|e| panic!("failed to read librustdoc/{rustdoc_css}: {e}"));
let noscript_css_contents = std::fs::read_to_string(librustdoc_path.join(noscript_css))
.unwrap_or_else(|e| panic!("failed to read librustdoc/{noscript_css}: {e}"));
compare_themes_from_files(
"light",
rustdoc_css_contents.lines().enumerate().map(|(i, l)| (i + 1, l.trim())),
noscript_css_contents.lines().enumerate().map(|(i, l)| (i + 1, l.trim())),
&mut check,
);
compare_themes_from_files(
"dark",
rustdoc_css_contents.lines().enumerate(),
noscript_css_contents.lines().enumerate(),
&mut check,
);
}
fn compare_themes_from_files<'a>(
name: &str,
mut rustdoc_css_lines: impl Iterator<Item = (usize, &'a str)>,
mut noscript_css_lines: impl Iterator<Item = (usize, &'a str)>,
check: &mut RunningCheck,
) {
let begin_theme_pat = format!("/* Begin theme: {name}");
let mut found_theme = None;
let mut found_theme_noscript = None;
while let Some((rustdoc_css_line_number, rustdoc_css_line)) = rustdoc_css_lines.next() {
if !rustdoc_css_line.starts_with(&begin_theme_pat) {
continue;
}
if let Some(found_theme) = found_theme {
check.error(format!(
"rustdoc.css contains two {name} themes on lines {rustdoc_css_line_number} and {found_theme}",
));
return;
}
found_theme = Some(rustdoc_css_line_number);
while let Some((noscript_css_line_number, noscript_css_line)) = noscript_css_lines.next() {
if !noscript_css_line.starts_with(&begin_theme_pat) {
continue;
}
if let Some(found_theme_noscript) = found_theme_noscript {
check.error(format!(
"noscript.css contains two {name} themes on lines {noscript_css_line_number} and {found_theme_noscript}",
));
return;
}
found_theme_noscript = Some(noscript_css_line_number);
compare_themes(name, &mut rustdoc_css_lines, &mut noscript_css_lines, check);
}
}
}
fn compare_themes<'a>(
name: &str,
rustdoc_css_lines: impl Iterator<Item = (usize, &'a str)>,
noscript_css_lines: impl Iterator<Item = (usize, &'a str)>,
check: &mut RunningCheck,
) {
let end_theme_pat = format!("/* End theme: {name}");
for (
(rustdoc_css_line_number, rustdoc_css_line),
(noscript_css_line_number, noscript_css_line),
) in rustdoc_css_lines.zip(noscript_css_lines)
{
if noscript_css_line.starts_with(":root, :root:not([data-theme]) {")
&& (rustdoc_css_line.starts_with(&format!(r#":root[data-theme="{name}"] {{"#))
|| rustdoc_css_line.starts_with(&format!(
r#":root[data-theme="{name}"], :root:not([data-theme]) {{"#
)))
{
// selectors are different between rustdoc.css and noscript.css
// that's why they both exist: one uses JS, the other uses media queries
continue;
}
if noscript_css_line.starts_with(&end_theme_pat)
&& rustdoc_css_line.starts_with(&end_theme_pat)
{
break;
}
if rustdoc_css_line != noscript_css_line {
check.error(format!(
r#"noscript.css:{noscript_css_line_number} and rustdoc.css:{rustdoc_css_line_number} contain copies of {name} theme that are not the same
- {noscript_css_line}
+ {rustdoc_css_line}"#,
));
return;
}
}
}
|
