Upgrade to Docsy 0.5.1 via NPM module (1.31 backport)

[sftim]

Based on PR #48812 by @chalin

Upgrade the v1.31 docs to use Docsy 0.5.1

[netlify]

✅ Deploy Preview for k8s-v1-31 ready!

Name	Link
🔨 Latest commit	89896bc
🔍 Latest deploy log	https://app.netlify.com/sites/k8s-v1-31/deploys/678e782ef48d4700078f0372
😎 Deploy Preview	https://deploy-preview-49031--k8s-v1-31.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[sftim]

/hold
for PR #48812 to merge

/area web-development

[netlify]

✅ Pull request preview available for checking

Built without sensitive environment variables

Name	Link
🔨 Latest commit	89896bc
🔍 Latest deploy log	https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/678e782e8548850008c1248e
😎 Deploy Preview	https://deploy-preview-49031--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[sftim]

Hmm. https://deploy-preview-49031--k8s-v1-31.netlify.app/community/ looks wrong and I cannot figure out why.

/retitle [WIP] Upgrade to Docsy 0.5.1 via NPM module (1.31 backport)

[shurup]

https://deploy-preview-49031--k8s-v1-31.netlify.app/community/ looks wrong

As I mentioned on Slack, the disappearing footer problem is solved if we remove the <section id="deprecation-warning"> block. However, after further investigation, I found out that not this specific block is a culprit, but having height: 100vh; for .td-outer in the very beginning of _main-container.scss. To be more precise, on some other pages, such as blog, this attribute s overwritten by _custom.scss (it defines height: auto; for .td-outer). Therefore, we need:

a) include _custom.scss on all other pages where it's missing now (they include /, /partners/, and /community/; I could not find any other);
b) OR remove height: 100vh; from the main file (personally, I don't find it too useful for the cases it covers. What it does is display the footer below your first screen for the pages whose content is not enough to fill your whole screen vertically).

[sftim]

/retitle Upgrade to Docsy 0.5.1 via NPM module (1.31 backport)

Should stay held until PR #48812 merges

[sftim]

/hold cancel

#48812 has merged

[sftim]

I put a workaround in place for the minimum page height, and all looks good.

[shurup]

The footer looks good in both my browsers now! 👍

However, the #deprecation-warning block displaying the 'Kubernetes v1.31 documentation is no longer actively maintained. [..]' message has width: 100vw (at assets/scss/_custom.scss:1129), which adds an unnecessary horizontal scroll for the /community page in Google Chrome. Changing it to width: 100% fixes the issue. Not sure whether it's relevant to this specific PR(?), so I didn't commit a fix here.

[sftim]

If we can save the fix to a follow-up PR, let's do that.

[shurup]

If we can save the fix to a follow-up PR, let's do that.

Absolutely! I checked this issue for the currently rendered website and realised it's here already (unrelated to this upgrade). Thus, created another PR to fix it: #49364

[chalin]

LGTM

Nice that this fixes the missing footers in the Partners and Community pages, e.g.:

I'm assuming others have checked the containerized build and serve.

/cc @nate-double-u

[chalin]

Anything holding this back @sftim et all?

[sftim]

This PR is waiting on reviewer capacity.

[nate-double-u]

I'm assuming others have checked the containerized build and serve.

I've spun this up in a container and it looks good. (With the caveat that I still need to do the api-ref-generator dance - but that's not an issue with this PR)

/lgtm

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: 90b91396d61bc5cc7b578a502975ced73addd9ed

[sftim]

containerized build and serve

This is likely to throw up #49460 - and I'm OK with that (people rarely work on the release-1.31 branch with a local preview).

[reylejano]

/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chalin, reylejano

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [reylejano]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment