Skip to content

feat: add tags footer at the end of user guide pages#840

Merged
notowen333 merged 3 commits into
strands-agents:mainfrom
notowen333:tag-pills
May 15, 2026
Merged

feat: add tags footer at the end of user guide pages#840
notowen333 merged 3 commits into
strands-agents:mainfrom
notowen333:tag-pills

Conversation

@notowen333
Copy link
Copy Markdown
Contributor

Description

Footer on every user guide page with tags pills. Each pill is clickable and shows all of the related pages.

Related Issues

n/a

Type of Change

  • Other (please describe):

Checklist

  • I have read the CONTRIBUTING document
  • My changes follow the project's documentation style
  • I have tested the documentation locally using npm run dev
  • Links in the documentation are valid and working

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 14, 2026

Documentation Preview Ready

Your documentation preview has been successfully deployed!

Preview URL: https://d3ehv1nix5p99z.cloudfront.net/pr-cms-840/docs/user-guide/quickstart/overview/

Updated at: 2026-05-15T19:13:46.124Z

zastrowm
zastrowm previously approved these changes May 15, 2026
Comment thread src/components/overrides/MarkdownContent.astro
Comment thread src/components/overrides/MarkdownContent.astro
Comment thread src/components/PageTags.astro
Comment thread src/util/related-docs.ts
Comment thread src/components/PageTags.astro Outdated
mehtarac
mehtarac previously approved these changes May 15, 2026
Addresses PR review:
- Anchor the expansion panel absolutely so opening/closing it no longer
  reflows the rest of the page. ESC and click-outside dismiss too.
- Rank pages within a tag panel by the same specificity-weighted Jaccard
  scorer used for headless Related Pages, instead of alphabetical.
- Drop the inline regex slugger in PageTags.astro in favor of the
  github-slugger import already used by content.config.ts.
@notowen333 notowen333 dismissed stale reviews from mehtarac and zastrowm via 8bbed47 May 15, 2026 16:01
Floating popover worked in the simple case but broke on long-tag pages:
a CSS height cap pinned the panel to the viewport but the surrounding
article scrolled instead of the panel, and on tall lists the panel
still overlapped pagination. Going back to the inline disclosure
(panel pushes content down) sidesteps both.

Tighten the matches list: line-height on the li (not the inline
anchor — that doesn't size the line box) so wrapped lines of long
titles pack closer than the gap between separate items.

Keeps the relevance ranking and github-slugger adoption from 8bbed47.
@notowen333 notowen333 merged commit 5c5fc96 into strands-agents:main May 15, 2026
4 of 5 checks passed
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.

3 participants