-
Notifications
You must be signed in to change notification settings - Fork 17
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
Support for JSON-LD #23
Comments
Hmm... special casing it vs just making it a Script tag probably goes beyond the scope of this project, keeping things minimal wrapper over native elements. You could always wrap it if we supported inserting script tags which we don't as of yet. I think we've collected a few reasons to do that now. So it's probably worth adding Script tags. Meta tags/Title have certain overriding characteristics and I guess Script would just insert until unmounted. Mostly haven't added it yet because the library this library is based on never did. Only thing I'm pondering is if limiting it in the proposed way provides some advantage over script. Ergonomics aside I wonder if there are security considerations of generic script insertion. |
@ryansolid exactly That's why I came up with Another idea would be just to export |
Perhaps I missed something, but it seems solid-meta doesn't currently expose a way to set json-ld on the head.
JSON-LD is Google's preferred schema markup and it's kind of a big deal for SEO.
I am currently trying out Solid Start and couldn't find a way to set it up, I was expecting we could do something like:
Anyway, I am about to open a PR that would allow us to do:
which, IMO, would be even more ergonomic.
I don't like the fact that
JSONLD
is all caps. Maybe we can find a better name.Please let us not choose:
JavaScriptObjectNotationLinkedData
😂The text was updated successfully, but these errors were encountered: