Merge pull request #223 from KubaO/staging

Help add-in hover help and a detachable help window, exe-run reporting in the harness, and 997 reproducibility bugs

Author: KubaO

.github/workflows/checks.yml

@@ -97,7 +97,11 @@ jobs:
         # broken link still produces a site worth inspecting -- so the
         # step fails on the exit code: 1 when the check found a problem,
         # 2 when the build could not run.
-        run: node builder/tbdocs.mjs --src docs --no-fetch-assets --check-audit-index
+        #
+        # --no-help-archive: a local build ends by zipping the offline tree
+        # for the IDE help add-in. CI has no use for that archive and must
+        # not spend time on it.
+        run: node builder/tbdocs.mjs --src docs --no-fetch-assets --no-help-archive --check-audit-index
       # The gates both workflows run, in one list: see
       # .github/actions/run-gates/action.yml, which check_ci_workflows.mjs
       # checks against test.bat and check.bat. Each gate is its own group in

.github/workflows/tbdocs-gh-pages.yml

@@ -92,7 +92,11 @@ jobs:
         # baseurl off the config it just built with. Only the
         # online tree gets a base path -- the offline tree's links are all
         # relative after the rewrite.
-        run: node builder/tbdocs.mjs --src docs --url '${{ steps.pages.outputs.origin }}' --baseurl '${{ steps.pages.outputs.base_path }}' --no-fetch-assets --check-audit-index
+        #
+        # --no-help-archive: a local build ends by zipping the offline tree
+        # for the IDE help add-in. CI has no use for that archive and must
+        # not spend time on it.
+        run: node builder/tbdocs.mjs --src docs --url '${{ steps.pages.outputs.origin }}' --baseurl '${{ steps.pages.outputs.base_path }}' --no-fetch-assets --no-help-archive --check-audit-index
       # The gates both workflows run, in one list: see
       # .github/actions/run-gates/action.yml, which check_ci_workflows.mjs
       # checks against test.bat and check.bat. checks.yml runs only on pull

.gitignore

@@ -28,8 +28,9 @@ indexer/.packages/
 # docs/assets/fonts/ -- only the archives are throwaway.
 /.font-cache/
 
-# The help add-in's documentation archive is built by scripts/build_help_archive.mjs
-# from docs/_site-offline, and is too large to commit.
+# The help add-in's documentation archive is written by a local build of the
+# documentation (builder/help-archive-step.mjs) from docs/_site-offline, and is
+# too large to commit.
 /add-in/Resources/HELP/
 
 # A bug reproducer's zip is only for attaching to its GitHub issue (see

BUGS-TO-REPORT.md

@@ -27,11 +27,12 @@ prefer that shape where the layout allows it.
 **A footnote back-link shows the hazard.** It is a bare inline `<a>` holding a
 single U+21A9 with no padding, so its target is whatever box the font gives that
 glyph: 12.09 x 19 on Segoe UI, 12.09 x 15 on Liberation Sans, 15.97 x 17 on
-Inter. **None of those meets 24 x 24.** It passed on axe's *spacing* exception
---- a 24px circle centred on the target clears its neighbours --- until Inter's
-arrow, 3.9px wider, moved the centre far enough to intersect one. The fix sizes
-the link outright (`display: inline-block; min-width/min-height: 24px`) rather
-than restoring the clearance.
+Inter. **None of those meets 24 x 24.** It can pass on axe's *spacing* exception
+--- a 24px circle centred on the target clears its neighbours --- and a wider
+glyph fails it: Inter's arrow is 3.9px wider than Segoe UI's, enough to move the
+centre far enough to intersect a neighbour. The link is sized outright
+(`display: inline-block; min-width/min-height: 24px`) rather than relying on the
+clearance.
 
 Two rules follow. **Run the full sweep after any change that moves type
 metrics**: the thirteen-page sample stays clean through such a change and would