Copy screenshots and reftests properly to website

Author: fschutt

.github/workflows/rust.yml

@@ -618,6 +618,27 @@ jobs:
           name: reftest-results
           path: artifacts-reftest
 
+      - name: Download Linux screenshots
+        uses: actions/download-artifact@v4
+        continue-on-error: true
+        with:
+          name: screenshots-ubuntu-22.04
+          path: artifacts-screenshots-linux
+
+      - name: Download macOS screenshots
+        uses: actions/download-artifact@v4
+        continue-on-error: true
+        with:
+          name: screenshots-macos-14
+          path: artifacts-screenshots-macos
+
+      - name: Download Windows screenshots
+        uses: actions/download-artifact@v4
+        continue-on-error: true
+        with:
+          name: screenshots-windows-2022
+          path: artifacts-screenshots-windows
+
       - name: Debug - Tree after download
         run: |
           sudo apt-get update && sudo apt-get install -y tree
@@ -669,6 +690,31 @@ jobs:
             fi
           fi
           
+          # Copy screenshots to website images directory
+          # Screenshots are named like "hello-world-linux.png" and need to be renamed to "hello-world.linux.png"
+          mkdir -p website/images
+          for screenshot_dir in artifacts-screenshots-linux artifacts-screenshots-macos artifacts-screenshots-windows; do
+            if [ -d "$screenshot_dir" ]; then
+              for png in "$screenshot_dir"/*.png; do
+                if [ -f "$png" ]; then
+                  filename=$(basename "$png")
+                  # Extract OS from directory name
+                  case "$screenshot_dir" in
+                    *linux*) os="linux" ;;
+                    *macos*) os="mac" ;;
+                    *windows*) os="windows" ;;
+                  esac
+                  # Convert "hello-world-linux.png" to "hello-world.linux.png"
+                  # The filename already has the OS in it, so we just rename
+                  newname=$(echo "$filename" | sed "s/-${os}/.${os}/")
+                  cp "$png" "website/images/$newname"
+                  echo "  [OK] Copied screenshot: $newname"
+                fi
+              done
+            fi
+          done
+          ls -la website/images/ || echo "No images directory"
+          
           # Update file sizes in release HTML
           # The HTML contains "N/A" or placeholder sizes that need to be replaced with real sizes
           if [ -d "$RELEASE_DIR" ]; then

doc/src/reftest/mod.rs

@@ -240,9 +240,62 @@ pub fn run_reftests(config: RunRefTestsConfig) -> anyhow::Result<()> {
 /// Generate a reftest HTML page without running any tests.
 /// If test_dir exists and contains tests, it will show "X tests found (not run)".
 /// If test_dir doesn't exist or is empty, it will show "0 tests found".
+/// 
+/// This function also copies existing reftest results and images from the standard
+/// reftest output directory (doc/target/reftest) if they exist.
 pub fn generate_reftest_page(output_dir: &Path, test_dir: Option<&Path>) -> anyhow::Result<()> {
     fs::create_dir_all(output_dir)?;
 
+    // Check if we have existing reftest results to copy
+    // The reftest command outputs to doc/target/reftest, so look for results there
+    let manifest_dir = Path::new(env!("CARGO_MANIFEST_DIR"));
+    let existing_reftest_dir = manifest_dir.join("target").join("reftest");
+    let existing_results_json = existing_reftest_dir.join("results.json");
+    let existing_index_html = existing_reftest_dir.join("index.html");
+    let existing_reftest_img = existing_reftest_dir.join("reftest_img");
+
+    // If we have existing results, copy them instead of generating empty page
+    if existing_results_json.exists() && existing_index_html.exists() {
+        println!("  Found existing reftest results, copying...");
+        
+        // Copy index.html
+        fs::copy(&existing_index_html, output_dir.join("index.html"))?;
+        println!("    Copied index.html");
+        
+        // Copy results.json
+        fs::copy(&existing_results_json, output_dir.join("results.json"))?;
+        println!("    Copied results.json");
+        
+        // Copy reftest_img directory if it exists
+        if existing_reftest_img.exists() && existing_reftest_img.is_dir() {
+            let dest_img_dir = output_dir.join("reftest_img");
+            fs::create_dir_all(&dest_img_dir)?;
+            
+            let mut img_count = 0;
+            for entry in fs::read_dir(&existing_reftest_img)? {
+                let entry = entry?;
+                let path = entry.path();
+                if path.is_file() {
+                    let filename = path.file_name().unwrap();
+                    fs::copy(&path, dest_img_dir.join(filename))?;
+                    img_count += 1;
+                }
+            }
+            println!("    Copied {} reftest images", img_count);
+        }
+        
+        // Also copy to reftest.html in parent directory
+        if output_dir.file_name().map(|n| n == "reftest").unwrap_or(false) {
+            if let Some(parent) = output_dir.parent() {
+                fs::copy(output_dir.join("index.html"), parent.join("reftest.html"))?;
+            }
+        }
+        
+        println!("  [OK] Reftest results copied from existing run");
+        return Ok(());
+    }
+
+    // No existing results, generate placeholder page
     // Count tests if directory provided
     let test_count = test_dir
         .and_then(|dir| find_test_files(dir).ok())

doc/src/reftest/report_template.html

@@ -272,15 +272,16 @@ <h1 style="display:none;">Azul GUI Framework</h1>
         <a href="https://azul.rs"><img src="https://azul.rs/logo.svg" alt="Azul Logo" style="width:100px;height:100px;"></a>
       </header>
       <nav>
-        <ul>
+        <ul class='nav-grid'>
           <li><a href='https://azul.rs'>overview</a></li>
-          <li><a href='https://azul.rs/releases'>releases</a></li>
+          <li><a href='https://azul.rs/releases.html'>releases</a></li>
           <li><a href='https://github.com/fschutt/azul'>code</a></li>
           <li><a href='https://discord.gg/V96ZGKqQvn'>discord</a></li>
-          <li><a href='https://azul.rs/guide'>guide</a></li>
-          <li><a href='https://azul.rs/api'>api</a></li>
-          <li class="active"><a href='https://azul.rs/reftest'>reftests</a></li>
-          <li><a href='https://azul.rs/donate'>donate</a></li>
+          <li><a href='https://azul.rs/guide.html'>guide</a></li>
+          <li><a href='https://azul.rs/api.html'>api</a></li>
+          <li class="active"><a href='https://azul.rs/reftest.html'>reftests</a></li>
+          <li><a href='https://azul.rs/blog.html'>blog</a></li>
+          <li><a href='https://azul.rs/donate.html'>donate</a></li>
         </ul>
       </nav>
     </aside>

examples/c/minimal_test.c

@@ -161,7 +161,6 @@ else
         "calc"
         "hello-world"
         "infinity"
-        "minimal_test"
         "opengl"
         "widgets"
         "xhtml"

scripts/build_all.sh

@@ -332,8 +332,7 @@ else
 
             # Skip examples that are not suitable for screenshots
             case "$example_name" in
-                minimal_test|infinity|opengl)
-                    # minimal_test: not a real example
+                infinity|opengl)
                     # infinity: runs indefinitely
                     # opengl: requires specific OpenGL setup
                     echo -e "${YELLOW}Skipping ${example_name} (not suitable for automated screenshots)${NC}"