Skip to content

refactor(tools): move methodology disclosure below the tool widget - #27

Merged
Bissbert merged 1 commit into
mainfrom
refactor/tools-details-below
May 12, 2026
Merged

Bissbert merged 1 commit into
mainfrom
refactor/tools-details-below

Conversation

@Bissbert

Copy link
Copy Markdown
Contributor

Summary

  • Moves the collapsible <details> SEO prose block from above the React tool widget to below it on all 7 tool pages (/tools, /tools/measurement, /tools/optical, /tools/lab, /tools/identification, /tools/advanced, /tools/conversions).
  • The h1 and lead paragraph remain at the top; only the <details> block relocates.
  • Replaces mb-6 with mt-12 on each <details> element for visual separation below the widget.

The prose content stays in the DOM and remains fully indexable by search engines. Users now land directly on the interactive tool rather than scrolling past a collapsed disclosure block to reach it.

Test plan

  • npm run build passes (912 pages, no errors)
  • Positional check on all 7 built pages: <details> character offset > astro-island character offset (confirmed for all 7)
  • Word counts all well above 300 (range: 522–1660)
  • aria-label="Tool categories" remains 0 occurrences on /tools/ (duplicate-nav fix not regressed)

Users land directly on the interactive widget without having to scroll
past a collapsed prose block. The <details> SEO disclosure remains in
the DOM after the widget, keeping all content indexable, but no longer
gates access to the tool on page load. mb-6 replaced with mt-12 for
adequate visual separation below the widget.
@github-actions

Copy link
Copy Markdown

🚀 Preview deployed to: https://a27572a3.gemmology-dev.pages.dev

@Bissbert
Bissbert merged commit c4dac32 into main May 12, 2026
3 of 4 checks passed

This branch was successfully deployed

1 active deployment
gemmology-dev (Preview) — 1ad18093 Deployed May 11, 2026 by github-actions[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant