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

adopt lushnikov's vanilla viewer? #184

Open
paulirish opened this issue Feb 10, 2022 · 1 comment
Open

adopt lushnikov's vanilla viewer? #184

paulirish opened this issue Feb 10, 2022 · 1 comment

Comments

@paulirish
Copy link
Member

stepping back from #183

I've always thought https://vanilla.aslushnikov.com/ is quite nice and competes well against this version. (I especially like that it needs very little maintenance).

The only reason I didn't adopt it wholesale a few years ago was SEO concerns. (cuz ours is SSG but vanilla is entirely clientside). At this point, i'm not sure there's a meaningful distinction between the SEO performance of the two sites.

two queries that are among our top 25 search queries:

indexing-wise they seem fairly equivalent with useful context in the snippet, but i'd probably give edge to vanilla since the page title matches the method name exactly. vanilla also navigates you directly to the correct scroll position. unexpected but it works!

for a more general and common search

here ours has the related links that i never see with vanilla. so we got an edge.

all in all, i'm interested in adopting his and living that maintenance free life, baybee!
curious for your thoughts @TimvdLippe

@TimvdLippe
Copy link
Contributor

I don't have particularly strong opinions on any of this. I enjoyed fiddling with the code, but forking sounds also good. @hashseed do you have thoughts on this?

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

No branches or pull requests

2 participants