-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Adding basic meta tags. #565
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
Conversation
himanshuc3
commented
Feb 22, 2018
- MIME type
- keywords
- Description
This is great! I might suggest avoiding the word "playground", I like the spirit of that very much, but it might be confusing as there is @yining1023's project which is actually called p5 playground! http://1023.io/p5-inspector/ I think for now: "p5.js", "p5.js web editor", "web editor" are good? |
I would also add "processing", "code editor" |
could you change the description from "A web editor for sketching using p5.js library." to "A web editor for p5.js, a JavaScript library with the goal of making coding accessible to artists, designers, educators, and beginners." |
⚡ |
could you also add these meta tags to |
Do you want me add the same or have a discussion about what their description should contain. |
i think they should be the same, thanks! |
@catarak I added meta tags to 404error page. MailLayout page is made using mjml library and I could not find any existing meta tag support, though basic tags such as IE-edge support, content type are added by default. Adding meta tags to mailLayout could be made using another PR as a custom component could be made for it. |
okay! i can open an issue about it. maybe emails don't need meta tags, but we can investigate it! |