Skip to content

Commit 5887662

Browse files
authored
Minimal style + JS code for better manual review of screenshot differences. (#8643)
1 parent b76e032 commit 5887662

File tree

3 files changed

+35
-6
lines changed

3 files changed

+35
-6
lines changed

pkg/pub_integration/tool/compare_screenshots.dart

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -110,23 +110,32 @@ class _CompareTool {
110110
File(p.join(_reportDir.path, relativeDir, '$basename-after.png'));
111111
await afterFile.writeAsBytes(afterBytes);
112112

113-
_report.writeln('`$path`\n');
113+
_report.writeln('\n<div class="image">');
114+
_report.writeln('\n### `$path`\n\n');
114115
_report.writeln(
115-
'![before](${p.join(relativeDir, '$basename-before.png')})\n');
116-
_report
117-
.writeln('![after](${p.join(relativeDir, '$basename-after.png')})\n');
116+
'- ![before](${p.join(relativeDir, '$basename-before.png')})\n');
117+
_report.writeln(
118+
'- ![after](${p.join(relativeDir, '$basename-after.png')})\n');
118119
_report
119-
.writeln('![diff](${p.join(relativeDir, '$basename-diff.png')})\n');
120+
.writeln('- ![diff](${p.join(relativeDir, '$basename-diff.png')})\n');
121+
_report.writeln('\n</div>');
120122
_report.writeln();
121123
}
122124

123125
await _writeIndexHtml();
124126
}
125127

126128
Future<void> _writeIndexHtml() async {
129+
final script = await File('tool/comparison_web.js').readAsString();
130+
final styles = await File('tool/comparison_web.css').readAsString();
127131
await File(p.join(_reportDir.path, 'index.html')).writeAsString([
128-
'<html><body>',
132+
'<html>',
133+
'<head>',
134+
'<style>\n$styles\n</style>',
135+
'</head>',
136+
'<body>',
129137
markdownToHtml(_report.toString()),
138+
'<script>\n$script\n</script>\n',
130139
'</body></html>',
131140
].join('\n'));
132141
}
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
.image ul {
2+
display: flex;
3+
flex-direction: column;
4+
list-style-type: none;
5+
}
6+
7+
.image ul li {
8+
margin: 1px;
9+
}
10+
11+
.image.row ul {
12+
flex-direction: row;
13+
}
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
const imageDivs = document.querySelectorAll('div.image');
2+
3+
imageDivs.forEach(function(div) {
4+
div.addEventListener('click', function() {
5+
this.classList.toggle('row');
6+
});
7+
});

0 commit comments

Comments
 (0)