fix: correct branding from 'kanvas.new' to 'Kanvas' and fix X/Twitter social preview #196
fix: correct branding from 'kanvas.new' to 'Kanvas' and fix X/Twitter social preview #196Deepika-s04 wants to merge 7 commits intomeshery-extensions:masterfrom
Conversation
- Updated baseURL and canonicalBaseURL in hugo.toml - Improved Open Graph and Twitter meta tags in layouts/partials/head.html Closes meshery-extensions#195 Signed-off-by: Deepika <deepika0119.becse24@chitkara.edu.in>
|
🚀 Preview deployment: https://meshery-extensions.github.io/kanvas-site/pr-preview/pr-196/
|
Signed-off-by: Deepika <deepika0119.becse24@chitkara.edu.in>
Signed-off-by: Deepika <deepika0119.becse24@chitkara.edu.in>
Signed-off-by: Deepika <deepika0119.becse24@chitkara.edu.in>
There was a problem hiding this comment.
Pull request overview
This PR updates the Hugo site configuration and <head> meta tags to correct product branding (“Kanvas”) and improve social sharing previews (Open Graph + Twitter Card metadata), aligning with issue #195.
Changes:
- Updates
hugo.tomlsite settings (branding/title, base URL configuration, description text). - Adjusts
layouts/partials/head.htmlmeta tags for Open Graph and Twitter Cards (title/description/image/url + canonical handling).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
layouts/partials/head.html |
Updates OG/Twitter meta tags and canonical/robots handling for improved previews/SEO. |
hugo.toml |
Updates site title/description and base/canonical URL settings intended to support correct branding and previews. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| description = "Infrastructure as Design for collaborative Cloud and Kubernetes management" | ||
|
|
||
| canonicalBaseURL = "https://www.kanvas.new" | ||
| canonicalBaseURL = "" |
| <meta property="og:title" content="{{ if .IsHome }}{{ .Site.Title }}{{ else }}{{ .Title }} | {{ .Site.Title }}{{ end }}" /> | ||
| <meta property="og:description" content="{{ with .Description }}{{ . }}{{ else }}{{ .Site.Params.description }}{{ end }}" /> | ||
| <meta property="og:type" content="website" /> | ||
| <meta property="og:url" content="{{ .Permalink }}" /> | ||
| <meta property="og:image" content="{{ .Site.BaseURL }}brand/kanvas/stacked/kanvas-stacked-color.png" /> | ||
|
|
||
| <meta name="twitter:card" content="summary_large_image"> | ||
| <meta name="twitter:title" content="{{ if .IsHome }}{{ .Site.Title }}{{ else }}{{ .Title }} | {{ .Site.Title }}{{ end }}"> | ||
| <meta name="twitter:description" content="{{ with .Description }}{{ . }}{{ else }}{{ .Site.Params.description }}{{ end }}"> | ||
| <meta name="twitter:image" content="{{ .Site.BaseURL }}brand/kanvas/stacked/kanvas-stacked-color.png"> | ||
| <meta name="twitter:url" content="{{ .Permalink }}"> | ||
|
|
||
| {{- with site.Params.canonicalBaseURL }} | ||
| <link rel="canonical" href="{{ . | strings.TrimRight "/" }}{{ $.RelPermalink }}"> |
| <meta name="twitter:card" content="summary_large_image"> | ||
| <meta name="twitter:title" content="{{ if .IsHome }}{{ .Site.Title }}{{ else }}{{ .Title }} | {{ .Site.Title }}{{ end }}"> | ||
| <meta name="twitter:description" content="{{ with .Description }}{{ . }}{{ else }}{{ .Site.Params.description }}{{ end }}"> | ||
| <meta name="twitter:image" content="{{ .Site.BaseURL }}brand/kanvas/stacked/kanvas-stacked-color.png"> | ||
| <meta name="twitter:url" content="{{ .Permalink }}"> |
Bhumikagarggg
left a comment
Bhumikagarggg
left a comment
There was a problem hiding this comment.
Please do minimal changes.
|
Hi @Bhumikagarggg, I have updated the PR with minimal changes. |
|
@Deepika-s04 Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |
|
@Deepika-s04 No response! I'll closing your PR |
|
@Bhumikagarggg I apologize for the inconvenience. I had already made the minimal changes you requested and was waiting for your review. |
|
@Deepika-s04 Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |
Not only adding your PR in meeting minutes, you have to be there in the meeting. |
| <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.5.2/css/all.min.css" integrity="sha512-SnH5WK+bZxgPHs44uWIX+LLJAJ9/2PkPKZ5QiAj6Ta86w+fsb2TkcmfRyVX3pBnMFcV7oQPJkl9QevSCWr3W6A==" crossorigin="anonymous" referrerpolicy="no-referrer" /> | ||
| {{ partial "head-css.html" . }} | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
|
|
There was a problem hiding this comment.
Why you added one more extra line
There was a problem hiding this comment.
Don't do this keep the changes mininal
| <link rel="manifest" href="/favicons/site.webmanifest"> | ||
| <link rel="mask-icon" href="/favicons/safari-pinned-tab.svg"> | ||
| <link rel="shortcut icon" href="/favicons/favicon.ico"> | ||
|
|
What is the problem?
The site was displaying "kanvas.new" instead of the correct product name "Kanvas" in browser tabs and search results. Additionally, sharing the site link on X (Twitter) was showing a half image with no description in the preview.
Fixes #195
What is the fix?
Updated the site configuration and meta tags to display correct branding:
Fixed baseURL, canonicalBaseURL, and title in hugo.toml
Added proper Open Graph meta tags — og:title, og:description, og:image
Added proper Twitter Card meta tags — twitter:card, twitter:title, twitter:description, twitter:image
Files Changed
hugo.toml
layouts/partials/head.html
Screenshots
After Fix — Browser tab now correctly shows "Kanvas"
Signed commits