nchammas opened a new pull request, #59547:
URL: https://github.com/apache/spark/pull/59547

   ### What changes were proposed in this pull request?
   
   Upgrade AnchorJS to v5.0.0, and update its config accordingly to maintain 
roughly the current production behavior.
   
   Additionally:
   - Move the AnchorsJS config to `global.html`, which matches where we include 
and configure other vendored libraries.
   - Remove the `anchors` global from `main.js`, which is no longer needed now 
that the config is handled outside of there.
   - Fix the included license for AnchorJS so it includes the copyright 
holder's name and year.
   
   ### Why are the changes needed?
   
   The current version of AnchorJS we are using is [more than 11 years 
old][old]. The releases since then have added more accessible heading links and 
more reliable fragment generation.
   
   [old]: https://github.com/bryanbraun/anchorjs/releases/tag/1.1.1
   
   This upgrade will also keep the versions in sync with the main website's 
version of AnchorJS: https://github.com/apache/spark-website/pull/700
   
   ### Does this PR introduce _any_ user-facing change?
   
   No, documentation behavior tweaks only.
   
   ### How was this patch tested?
   
   I built and reviewed the site locally with `SKIP_API=1 bundle exec jekyll 
serve`. Confirmed that anchors work as expected, including for `h1` headings.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   I wrote this with assistance from GitHub Copilot.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to