Skip to content

content: Clarify sort order with PAGE.Rotate#3408

Merged
jmooring merged 1 commit intogohugoio:masterfrom
jmooring:rotate-sort-order
Feb 21, 2026
Merged

content: Clarify sort order with PAGE.Rotate#3408
jmooring merged 1 commit intogohugoio:masterfrom
jmooring:rotate-sort-order

Conversation

@jmooring
Copy link
Member

@jmooring jmooring commented Feb 21, 2026

https://deploy-preview-3408--gohugoio.netlify.app/methods/page/rotate/

This adds some detail (fallback) to the sort order, and provides an introductory sentence to each example.

@jmooring jmooring requested a review from bep February 21, 2026 21:55
@bep
Copy link
Member

bep commented Feb 21, 2026

I think this is great and don't hesitate to merge it.

As a side note, the natural thing would be to write:

{{ range .Rotate "language" }}
    <h2><a href="{{ .RelPermalink }}">{{ .LinkTitle }}</a></h2>
{{ end }}

@jmooring
Copy link
Member Author

the natural thing

I have no idea why I wrote it that way. In my defense, I'm pretty sure it was well after midnight.

@jmooring jmooring merged commit 25ce893 into gohugoio:master Feb 21, 2026
6 checks passed
@jmooring jmooring deleted the rotate-sort-order branch February 21, 2026 23:38
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.

2 participants