Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the Node/Jest toolchain (likely for security/maintenance), adds a new Man7.org userscript with a demo + documentation, and enhances the Amazon Subscribe & Save userscript to display estimated savings.
Changes:
- Bump devDependencies (Jest + jsdom) and refresh
package-lock.json. - Add
Man7.org/man7.user.jsplus a demo HTML page and a README describing the script. - Update Amazon Subscribe & Save script to also compute and display savings.
Reviewed changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updates Jest/JSDOM versions in devDependencies. |
| package-lock.json | Lockfile refresh to match updated dependency graph. |
| Man7.org/man7.user.js | New userscript: builds an options TOC, adds filtering, and collapses long sections. |
| Man7.org/demo/hexdump.1.html | Demo page that loads the Man7 userscript. |
| Man7.org/README.md | Documentation and links for the Man7 scripts and demos. |
| Amazon/AmazonSubscriptionTotal.user.js | Adds savings calculation/output alongside total cost. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| </head> | ||
|
|
||
| <body> | ||
| <script async src="../man7.user.js"></script> |
There was a problem hiding this comment.
This demo loads the userscript with async near the top of <body>, so it may execute before the man page DOM (e.g., the OPTIONS heading) exists, causing null dereferences in the script. Use defer, move the script tag to the end of the document, or wrap the userscript entrypoint in DOMContentLoaded.
| <script async src="../man7.user.js"></script> | |
| <script defer src="../man7.user.js"></script> |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Moved demo pages to new folder to be served by Github Pages * Switched to local copies of two css files used by demo pages to avoid hotlinking * Removed tracking scripts from the demo pages
Also added support for a third DOM structure
No description provided.