Skip to content

docs: refine benchmark artifacts management and add chart styles/scripts #127

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
TKorr merged 1 commit into from
Apr 29, 2026
Merged

docs: refine benchmark artifacts management and add chart styles/scripts #127

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 18 additions & 3 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,21 @@ out/
.DS_Store
/dhat-heap.json

# Benchmark artifacts
docs/benchmarks/latest/
docs/benchmarks/*/
# Benchmark artifacts.
#
# `docs/benchmarks/<version>/` mixes published Pages assets (which must be
# committed so GitHub Pages can serve them) with raw run artifacts (which
# must not). We exclude *files inside* each version dir, then explicitly
# re-include the five files that `bench-support::render_docs` produces
# and that `charts.html` references at runtime — the page 404s without
# `charts.js` / `charts.css`.
#
# Pattern note: `docs/benchmarks/*/*` (file-level) is required instead of
# `docs/benchmarks/*/` (directory-level) because git refuses to re-include
# files whose parent directory is excluded.
docs/benchmarks/*/*
!docs/benchmarks/*/charts.html
!docs/benchmarks/*/charts.css
!docs/benchmarks/*/charts.js
!docs/benchmarks/*/index.md
!docs/benchmarks/*/results.json
182 changes: 182 additions & 0 deletions docs/benchmarks/latest/charts.css
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,182 @@
/*!
* charts.css — sibling stylesheet for charts.html.
* Sized to run under `style-src 'self'` with no 'unsafe-inline'.
*
* Coordinated tokens with charts.html:
* - `.no-js #loading { display: none; }` cooperates with the parser-
* blocking inline script in <head>, which strips the `no-js` class
* before <body> paints so JS-enabled users immediately see the
* "Loading…" placeholder. JS-disabled users keep the class and the
* <noscript> block carries the "JavaScript is required" messaging.
* - `.hidden { display: none; }` is the class-based replacement for
* `style="display: none"` attributes on #error and #content; the JS
* path toggles via `classList`/`style.display` (DOM mutation, which
* CSP does not gate).
*/

* {
margin: 0;
padding: 0;
box-sizing: border-box;
}

body {
font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, sans-serif;
line-height: 1.6;
color: #333;
background: #f5f5f5;
padding: 20px;
}

.container {
max-width: 1400px;
margin: 0 auto;
background: white;
padding: 30px;
border-radius: 8px;
box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
}

h1 {
color: #2c3e50;
margin-bottom: 10px;
font-size: 2em;
}

.metadata {
background: #f8f9fa;
padding: 15px;
border-radius: 4px;
margin-bottom: 30px;
font-size: 0.9em;
}

.metadata-grid {
display: grid;
grid-template-columns: repeat(auto-fit, minmax(200px, 1fr));
gap: 10px;
}

.metadata-item {
display: flex;
gap: 5px;
}

.metadata-label {
font-weight: 600;
color: #555;
}

.chart-section {
margin-bottom: 40px;
}

.chart-section h2 {
color: #34495e;
margin-bottom: 15px;
padding-bottom: 10px;
border-bottom: 2px solid #3498db;
}

.chart-container {
position: relative;
height: 400px;
margin-bottom: 20px;
}

.chart-description {
color: #666;
font-size: 0.9em;
margin-bottom: 15px;
font-style: italic;
}

.links {
margin-top: 30px;
padding-top: 20px;
border-top: 1px solid #ddd;
display: flex;
gap: 20px;
flex-wrap: wrap;
}

.links a {
color: #3498db;
text-decoration: none;
padding: 8px 16px;
border: 1px solid #3498db;
border-radius: 4px;
transition: all 0.3s;
}

.links a:hover {
background: #3498db;
color: white;
}

.error {
background: #fee;
color: #c33;
padding: 20px;
border-radius: 4px;
border-left: 4px solid #c33;
white-space: pre-line;
font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, sans-serif;
}

.error code {
font-family: ui-monospace, SFMono-Regular, Menlo, Consolas, monospace;
background: rgba(0, 0, 0, 0.05);
padding: 1px 4px;
border-radius: 3px;
}

.loading {
text-align: center;
padding: 40px;
color: #666;
}

.chart-empty {
display: flex;
align-items: center;
justify-content: center;
height: 100%;
color: #888;
font-style: italic;
background: #fafafa;
border: 1px dashed #ddd;
border-radius: 4px;
padding: 20px;
text-align: center;
}

/* Class-based hiding for elements that start invisible (#error, #content);
pairs with `style-src 'self'` since `style="display: none"` would require
'unsafe-inline'. JS toggles via element.style.display (a DOM property
mutation, which CSP does not gate). */
.hidden {
display: none;
}

/* When scripting is disabled, hide the misleading "Loading..." placeholder
and let the <noscript> error block carry the message. The parser-blocking
inline script in charts.html's <head> removes `no-js` from <html> before
<body> paints, so JS-enabled users never see this rule applied. */
.no-js #loading {
display: none;
}

@media (max-width: 768px) {
.container {
padding: 15px;
}

h1 {
font-size: 1.5em;
}

.chart-container {
height: 300px;
}
}
Loading
Loading