Skip to content

Improve browsing of release notes and try.squeak.org integration - #70

Merged
LinqLover merged 7 commits into
masterfrom
release-notes-squeakjs
Aug 2, 2026
Merged

Improve browsing of release notes and try.squeak.org integration#70
LinqLover merged 7 commits into
masterfrom
release-notes-squeakjs

Conversation

@LinqLover

@LinqLover LinqLover commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Warning

Breaking change: The old numbered text anchors like #opensmalltalk-vm-100 for ordered collection items are no longer available but have been replaced by unnumbered annchors like #opensmalltalk-vm. For most affected pages such as /community or /features, this does not really matter as these pages are effectively so short that the relevant item will appear immediately anyway. For /documentation and /downloads, this might be most noticable as these pages are longer. However, these anchors have been never stable anyway, because every addition of a newer book or version of Squeak in the past has changed the anchors anyway. Offering backwards compatibility seems pointless for this reason.

image

Depends on squeak-smalltalk/try.squeak.org#2.

Note: I'm mainly creating these PRs for structuring my changes, for running CI & Copilot reviews on them, and for transparency. I don't really need or are waiting for a review, even though feedback is always welcome :)

…ks and their limitations and refers to the download page and try.squeak.org

Depends on squeak-smalltalk/try.squeak.org#2.
- For collection items, do not include order number in theeir ID. This is required for the previous commit, which requires stable download links. Titles should be unique anyway.
- Use modern scroll-margin-top instead of auxiliary anchor divs
…ueak on try.squeak.org

Also open links in demo mode of try.squeak.org and navigate to release notes in support images.
@LinqLover LinqLover self-assigned this Jul 11, 2026
Copilot AI review requested due to automatic review settings July 11, 2026 18:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves on-site browsing of release notes and strengthens integration with try.squeak.org by modernizing anchor behavior (moving ids onto headings) and adding a dedicated release-note header with “Download” / “Try in browser” actions.

Changes:

  • Replace legacy <span class="anchor"> anchors with element id anchors and a CSS scroll-margin-top offset.
  • Add a release note header include + new release_note layout and set it as the default layout for the release_notes collection.
  • Update the code: URL rewriter to generate try.squeak.org URLs with updated parameters and version-specific behavior.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
index.html Moves section anchors onto headings for home page sections.
_sass/custom.scss Replaces .anchor offset hack with scroll-margin-top and adds release-note header styling.
_release_notes/_note_head.html Adds release note header content and try.squeak.org / downloads actions.
_plugins/code_links_rewriter.rb Updates code: link rewriting to new try.squeak.org URL format and version-dependent behavior.
_layouts/section.html Moves category anchors onto <h2> ids.
_layouts/release_note.html New release-note layout that injects the release-note header above content.
_includes/item_with_img.html Moves per-item anchors onto <h3> ids (image/icon items).
_includes/item_text_only.html Moves per-item anchors onto <h3> ids (text-only items) and tweaks “Read on” link text.
_config.yml Switches release notes default layout from default to release_note.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread index.html Outdated
Comment thread _includes/item_text_only.html
Comment thread _includes/item_with_img.html
Comment thread _plugins/code_links_rewriter.rb
@LinqLover
LinqLover merged commit f1b4b57 into master Aug 2, 2026
2 checks passed
@LinqLover
LinqLover deleted the release-notes-squeakjs branch August 2, 2026 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants