feat(sites): add landing page for devsy.sh root#631
Conversation
Add a self-contained static landing page served at www.devsy.sh/ and remove the root redirect that previously sent visitors straight to the docs. The page reuses the docs brand (purple, Inter, logo mark) and Devcontainer-standard messaging. Sections: hero ("Ship from day zero"), a six-tile feature grid, "deploy anywhere" targets with Docker/Kubernetes/cloud/SSH icons, a three-step "how it works" stepper, and a download CTA.
✅ Deploy Preview for devsydev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
✅ Deploy Preview for images-devsy-sh canceled.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
sites/docs-devsy-sh/public/index.html (1)
561-567: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueGitHub link reuses
nav-linksclass with inline style override.Using
class="nav-links"on a single anchor appliesdisplay: flex,gap: 30px, and other nav-container properties that are meaningless for one link, thenstyle="display: inline-flex"overrides the display. This works but is fragile and confusing. A dedicated class or reusingbtn-ghostwould be clearer.♻️ Suggested refactor
- <a - class="nav-links" - href="https://github.com/devsy-org/devsy" - style="display: inline-flex" - aria-label="GitHub" - >GitHub</a - > + <a + class="nav-link-single" + href="https://github.com/devsy-org/devsy" + aria-label="GitHub" + >GitHub</a + >Add a small style:
.nav-link-single { font-size: 15px; font-weight: 500; color: var(--slate); } .nav-link-single:hover { color: var(--ink); }🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@sites/docs-devsy-sh/public/index.html` around lines 561 - 567, The GitHub anchor is using the shared nav container styling in a way that’s only meant for grouped links, plus an inline display override. Update the GitHub link in the footer/header markup to use a dedicated single-link class (or the existing btn-ghost pattern) instead of nav-links, and move the visual styling into the stylesheet so the anchor is styled consistently without inline overrides.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@sites/docs-devsy-sh/public/index.html`:
- Around line 174-178: The mobile breakpoint hides the entire navigation in
`.nav-links`, leaving no way to reach key header pages on small screens. Update
the header in `public/index.html` so `nav-links` is not the only path on mobile:
either add a simple mobile toggle/drawer using the existing header structure, or
keep a persistent Docs link visible in the `.nav-cta` group alongside the
current call-to-action items. Make sure the header remains navigable at the
max-width 820px breakpoint and verify the same fallback applies where the
related nav styles are defined.
- Around line 25-26: The Open Graph and Twitter metadata in the page head use
the www host, but the canonical site URL in Docusaurus is non-www, so align the
metadata domains with the canonical host used by the site configuration. Update
the head entries in the index page so og:url and any og:image/twitter:image
values match the same host as docusaurus.config.js, or ensure a consistent
redirect strategy between www and non-www.
---
Nitpick comments:
In `@sites/docs-devsy-sh/public/index.html`:
- Around line 561-567: The GitHub anchor is using the shared nav container
styling in a way that’s only meant for grouped links, plus an inline display
override. Update the GitHub link in the footer/header markup to use a dedicated
single-link class (or the existing btn-ghost pattern) instead of nav-links, and
move the visual styling into the stylesheet so the anchor is styled consistently
without inline overrides.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: da5963f3-a718-4955-8353-1118e6e0c1f6
📒 Files selected for processing (2)
sites/docs-devsy-sh/public/_redirectssites/docs-devsy-sh/public/index.html
💤 Files with no reviewable changes (1)
- sites/docs-devsy-sh/public/_redirects
| <meta property="og:url" content="https://www.devsy.sh/" /> | ||
| <meta property="og:image" content="https://www.devsy.sh/docs/media/devsy.png" /> |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
OG/Twitter URL uses www.devsy.sh but Docusaurus config uses devsy.sh (no www).
The Docusaurus config (docusaurus.config.js) sets url: "https://devsy.sh" without www, while this page's og:url and og:image/twitter:image use https://www.devsy.sh/. This inconsistency can cause search engines and social platforms to treat the two as non-canonical, splitting link equity and producing mismatched card previews.
Align the OG/Twitter URLs with the canonical domain used in the Docusaurus config, or add a www → non-www redirect (or vice versa) at the hosting layer.
🔗 Proposed fix
- <meta property="og:url" content="https://www.devsy.sh/" />
- <meta property="og:image" content="https://www.devsy.sh/docs/media/devsy.png" />
+ <meta property="og:url" content="https://devsy.sh/" />
+ <meta property="og:image" content="https://devsy.sh/docs/media/devsy.png" />And similarly for the Twitter image:
- content="https://www.devsy.sh/docs/media/devsy.png"
+ content="https://devsy.sh/docs/media/devsy.png"🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@sites/docs-devsy-sh/public/index.html` around lines 25 - 26, The Open Graph
and Twitter metadata in the page head use the www host, but the canonical site
URL in Docusaurus is non-www, so align the metadata domains with the canonical
host used by the site configuration. Update the head entries in the index page
so og:url and any og:image/twitter:image values match the same host as
docusaurus.config.js, or ensure a consistent redirect strategy between www and
non-www.
| @media (max-width: 820px) { | ||
| .nav-links { | ||
| display: none; | ||
| } | ||
| } |
There was a problem hiding this comment.
🩺 Stability & Availability | 🟠 Major | ⚡ Quick win
Mobile users lose header navigation with no alternative menu.
At max-width: 820px the .nav-links container is set to display: none, hiding the Features, Deploy anywhere, How it works, and Docs links. There is no hamburger menu, drawer, or fallback navigation. Mobile users can only access "Get Started" and "GitHub" from the header — the Docs link is unreachable until the footer.
Consider adding a simple toggle menu or at minimum a "Docs" link in the .nav-cta group that remains visible on mobile.
📱 Suggested mobile nav fallback
`@media` (max-width: 820px) {
.nav-links {
- display: none;
+ display: none;
}
}
+
+ `@media` (max-width: 820px) {
+ .nav-cta .mobile-docs {
+ display: inline-flex;
+ }
+ }
+
+ .mobile-docs {
+ display: none;
+ }Then add a persistent Docs link in the nav-cta group:
<div class="nav-cta">
+ <a class="nav-links mobile-docs" href="/docs/what-is-devsy">Docs</a>
<a
class="nav-links"
href="https://github.com/devsy-org/devsy"
style="display: inline-flex"
aria-label="GitHub"
>GitHub</a
<a class="btn btn-primary" href="/docs/getting-started/install"
>Get Started</a
</div>Also applies to: 554-559
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@sites/docs-devsy-sh/public/index.html` around lines 174 - 178, The mobile
breakpoint hides the entire navigation in `.nav-links`, leaving no way to reach
key header pages on small screens. Update the header in `public/index.html` so
`nav-links` is not the only path on mobile: either add a simple mobile
toggle/drawer using the existing header structure, or keep a persistent Docs
link visible in the `.nav-cta` group alongside the current call-to-action items.
Make sure the header remains navigable at the max-width 820px breakpoint and
verify the same fallback applies where the related nav styles are defined.
Surface Docs and GitHub in the header CTA group at the mobile breakpoint, where .nav-links is hidden. Replace the GitHub anchor's reuse of .nav-links plus an inline display override with a dedicated .nav-link-single class.
1 participant
Summary
Adds a self-contained static landing page served at the
www.devsy.sh/root, and removes the redirect that previously sent the root straight to/docs/what-is-devsy. The page is deployed by the existing docs Netlify build, which copiespublic/*to the site root.sites/docs-devsy-sh/public/index.html— new landing page. Single file, inline CSS, no framework or build step. Reuses the docs brand (primary#7C5CFF, Inter, the ring logo mark) and the Devcontainer-standard messaging.sites/docs-devsy-sh/public/_redirects— drop the/ → /docs/what-is-devsyrule so the root serves the new page./docs → /docs/what-is-devsyis unchanged.Summary by CodeRabbit
New Features
Bug Fixes
/docsroute still points to the documentation landing page.