-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
refactor(blog-plugin): blog archive reverse ordering of posts #9438
Conversation
cc @johnnyreilly, was also wondering if the items inside a year should also be in reverse order? |
✅ [V2]
To edit notification comments on pull requests, go to your Netlify site configuration. |
⚡️ Lighthouse report for the deploy preview of this PR
|
Size Change: 0 B Total Size: 1.13 MB ℹ️ View Unchanged
|
I actually moved away from the in-built docusaurus archive that was based on my blog post https://johnnyreilly.com/blog-archive-for-docusaurus because the ordering seemed strange to me. My handrolled approach looks like this: It looks like the changes you're suggesting align with what I'm doing and so (unsurprisingly) I think this is a good idea! Interestingly, I was advised it is better for SEO also. The idea being that you typically want newer blog posts to get more noticed. And the higher up the screen, the more significant a link is regarded. Unrelated to the change you're making here, is there a way to control the URL of the archive functionality? I think it's hardcoded to |
ok so let's merge this as is for now 👍 |
Motivation
Chronological ordering is confusing.
v3 is the best time to do this change.
I'm considering it as an UX improvement, not a breaking change.
Test Plan
preview
Test links
https://deploy-preview-9438--docusaurus-2.netlify.app/blog/archive
Related issues/PRs
johnnyreilly/blog.johnnyreilly.com#327
#5428