883081 - meta tagging not working from inside Thimble

Status: RESOLVED WONTFIX

(Webmaker Graveyard :: webmaker.org, defect)

Description

[Laura Hilliger [:epilepticrabbit]]

In my thimble page I have:

<meta name="webmaker:thumbnail" content="http://hivenyc.org/images/thumbnails_webmaker/hackjam.jpg">
<meta name="webmaker:description" content="A Hack Jam is great for teaming up people with different skill-sets to explore an interesting challenge, collaboratively building something new or improving something that already exists, learning and teaching new hacking skills.">
<meta name="webmaker:tags" content="guide">
<meta name="webmaker:tags" content="featured">

but the only thing that displays in the admin backend is the title and thumbnail. Description and tags have to be pasted directly into the backend.

Comment 1

[Laura Hilliger [:epilepticrabbit]]

hmm...tags printed this last make https://mozteach.makes.org/thimble/how-to-host-a-hive-popup. It just printed "guide,featured" but the webmaker: is needed for that to show up on /teach.

Comment 2

[Scott [:thecount] Downe]

Unfortunatly, webmaker:description is actually just descriptions, example:

<meta name="description" content="asdf">

This is because description is an actual meta tag recognized by more than just us, and not something we made ourselves. Anything with "webmaker:" is something we've made up.

This is obviously not obvious, and we should fix it.

Also, not yet sure why your tags are not being parsed but I'll look into it!

Comment 3

[Scott [:thecount] Downe]

Attachment: https://github.com/mozilla/thimble.webmaker.org/pull/124

This should help reduce confusion as to webmaker: in thimble tags.

If webmaker:description is used in a meta name, and it has because it makes complete sense to do this.

Hopefully this helps.

Comment 4

[Pomax]

this looks sane to me, but chris should have a more authoritative opinion

Comment 5

[Chris DeCairos (:cade)]

Comment on attachment 762685 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/124

It's good, R+

lets make sure we communicate this tagging update to all the content folks.