Skip to content

Jekyll docs template typo - All pages show "Deployment"#9548

Merged
jekyllbot merged 2 commits intojekyll:masterfrom
velle:page_title_bug
Feb 13, 2024
Merged

Jekyll docs template typo - All pages show "Deployment"#9548
jekyllbot merged 2 commits intojekyll:masterfrom
velle:page_title_bug

Conversation

@velle
Copy link
Copy Markdown
Contributor

@velle velle commented Feb 12, 2024

This is a 🐛 bug fix.

Summary

  • 1st commit: fixed typo in docs_contents_mobile.html: page -> current_page
  • 2nd commit: renamed current_page to item_page in docs_contents_mobile.html and docs_contents.html

I did the two fixes above as separate commits. This will be my very first PR, so I am not sure if you can see that I did two commits and see the description for each of them.

I would very much appreciate if you let me do if I did things the right way with this PR :)

Context

@mattr-
Copy link
Copy Markdown
Member

mattr- commented Feb 13, 2024

Thanks for fixing this! It's a great PR. ❤️

@jekyllbot: merge +docs

@jekyllbot jekyllbot merged commit 22c756a into jekyll:master Feb 13, 2024
jekyllbot added a commit that referenced this pull request Feb 13, 2024
github-actions bot pushed a commit that referenced this pull request Feb 13, 2024
velle: Jekyll docs template typo - All pages show "Deployment" (#9548)

Merge pull request 9548
monfresh pushed a commit to monfresh/jekyll that referenced this pull request Jun 5, 2024
monfresh pushed a commit to monfresh/jekyll that referenced this pull request Jun 5, 2024
@jekyll jekyll locked and limited conversation to collaborators May 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue with highlighting load tags for Django code

3 participants