feat: add data-ol-link-track click tracking to Follow and reading log buttons

[Copilot]

Summary

Adds data-ol-link-track attributes to the Follow button and the three reading-log shelf buttons (Want to Read, Currently Reading, Already Read) throughout the site, consistent with how other interactive elements (downloads, search sorts, CTAs) are already tracked.

Changes

Commit 1 — Follow buttons (Follow|<Context>)

• macros/Follow.html — added optional link_track parameter; a link_track_attr variable is computed once and applied to both the submit <button> (logged-in users) and the login-redirect <a> (logged-out users)
• templates/type/user/view.html → Follow|PatronPage
• templates/account/view.html (My Books / reading-log pages) → Follow|MyBooksPage
• templates/account/follows.html (Follows management page) → Follow|FollowsPage
• templates/lists/list_follow.html (lists carousel on book pages) → Follow|BookPage

Commit 2 — Reading log shelf buttons (ReadingLog|<Action>)

• templates/my_books/primary_action.html — sets a track_value conditional on current read state:
  • No status (logged in) → ReadingLog|WantToRead
  • Status 2 → ReadingLog|CurrentlyReading
  • Status 3 → ReadingLog|AlreadyRead
  • No work / logged out → ReadingLog|AddToList
• templates/my_books/dropdown_content.html — explicit tracking on each button in the shelf dropdown:
  • ReadingLog|RemoveFromShelf
  • ReadingLog|WantToRead
  • ReadingLog|CurrentlyReading
  • ReadingLog|AlreadyRead

Evidence

Tracking values follow the existing Category|Action pattern used across the codebase (e.g. CTAClick|Edit, Download|epub_ia, MyBooksSidebar|WantToRead).

The link_track parameter on Follow.html is optional (defaults to '') so all existing callers continue to work without modification.

[Copilot]

Pull request overview

Adds consistent data-ol-link-track analytics attributes to Follow CTAs and Reading Log shelf buttons across key templates, aligning with the existing Category|Action tracking pattern used elsewhere in Open Library templates/macros.

Changes:

• Extend macros/Follow.html with an optional link_track parameter and apply it to both logged-in (submit button) and logged-out (login redirect link) states.
• Add data-ol-link-track tracking to the Reading Log primary action button with a computed track_value based on current shelf/read state.
• Add explicit tracking attributes to each Reading Log dropdown shelf action (including Remove From Shelf).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
openlibrary/macros/Follow.html	Adds optional link_track param and emits data-ol-link-track on Follow button/link.
openlibrary/templates/type/user/view.html	Supplies `Follow
openlibrary/templates/account/view.html	Supplies `Follow
openlibrary/templates/account/follows.html	Supplies `Follow
openlibrary/templates/lists/list_follow.html	Supplies `Follow
openlibrary/templates/my_books/primary_action.html	Adds computed tracking value to the primary reading log action button.
openlibrary/templates/my_books/dropdown_content.html	Adds per-action tracking to reading log dropdown shelf buttons.

[Copilot]

link_track_attr interpolates link_track directly into an HTML attribute without escaping. If link_track ever contains ", <, etc (e.g. from a future caller passing non-constant data), this can break markup and become an attribute-injection/XSS vector. Consider escaping the value (e.g. via websafe()/web.net.htmlquote()) or avoiding string-built attributes by conditionally rendering data-ol-link-track="..." with an escaped $(...) value.