Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feat] Add base support of td-content-after-header hook #2192

Merged

Conversation

chalin
Copy link
Collaborator

@chalin chalin commented Feb 10, 2025

Important

This preliminary core feature implementation is experimental.


  • Adds a "render hook" for page content right after the td-content <header>, and so just before the page .Content.
  • Adds nothing more so that projects can experiment with use of site/page params etc. Docsy isn't ready yet to commit to richer solutions for Add a feature to warn readers that a blog post may be outdated. #1773 and the like.
  • The base hook template is empty.
  • Processes the template using .Render, which means that template lookup is based on content type.

@chalin chalin added the enhancement New feature or request label Feb 10, 2025
@chalin chalin force-pushed the chalin-im-td-content-after-header-hook-2025-02-10 branch from 9e56023 to 343439a Compare February 10, 2025 16:25
@chalin chalin merged commit 2239156 into google:main Feb 10, 2025
11 checks passed
@chalin chalin deleted the chalin-im-td-content-after-header-hook-2025-02-10 branch February 10, 2025 20:30
@chalin chalin added this to the 25Q1 milestone Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants