Skip to content

Conversation

@Caesarsage
Copy link
Contributor

Description

The "kubeadm upgrade node phase" section tabs were not updating content when clicked due to a name collision with the "kubeadm upgrade apply phase" section. Both tab groups were using the same name attribute, causing unexpected behavior.

Changes:
Changed the tab name for the "kubeadm upgrade node phase" section from name="tab-upgrade-phase" to name="tab-node-phase" to ensure unique tab group identifiers
This ensures each tab group has a unique name and prevents state sharing between different tab sections

Issue

Closes: #51358

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 23, 2025
@k8s-ci-robot k8s-ci-robot added language/en Issues or PRs related to English language size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jun 23, 2025
@netlify
Copy link

netlify bot commented Jun 23, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 82d573e
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/6859d2c1f1898300087a6dce
😎 Deploy Preview https://deploy-preview-51362--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@shurup
Copy link
Member

shurup commented Jun 24, 2025

Thanks for this fix! Works as expected now.

The deployed preview page is here.

/area web-development
/label tide/merge-method-squash
/lgtm

@k8s-ci-robot k8s-ci-robot added area/web-development Issues or PRs related to the kubernetes.io's infrastructure, design, or build processes tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. labels Jun 24, 2025
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 24, 2025
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 1f4c26a52a8f4568c6857ac1bef7d6491d9d73e0

Copy link
Member

@dipesh-rawat dipesh-rawat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dipesh-rawat

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:

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 24, 2025
@k8s-ci-robot k8s-ci-robot merged commit 92ef3b1 into kubernetes:main Jun 24, 2025
6 checks passed
ajaysundark pushed a commit to ajaysundark/website that referenced this pull request Jun 29, 2025
* chore: removing whitespaces

* chore: clean up whitespace in training.css

* Fix tab name collision in kubeadm upgrade phase documentation
rata pushed a commit to rata/kubernetes.github.io that referenced this pull request Jul 1, 2025
* chore: removing whitespaces

* chore: clean up whitespace in training.css

* Fix tab name collision in kubeadm upgrade phase documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/web-development Issues or PRs related to the kubernetes.io's infrastructure, design, or build processes cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Subphase navigation issue under kubeadm-upgrade-phase documentation .

4 participants