Skip to content

fix(web): resolve DOM nesting warnings and broken doc links - #625

Open
prakharsingh-74 wants to merge 1 commit into
containers:mainfrom
prakharsingh-74:fix/dom-nesting-and-docs-link
Open

fix(web): resolve DOM nesting warnings and broken doc links#625
prakharsingh-74 wants to merge 1 commit into
containers:mainfrom
prakharsingh-74:fix/dom-nesting-and-docs-link

Conversation

@prakharsingh-74

@prakharsingh-74 prakharsingh-74 commented Aug 3, 2026

Copy link
Copy Markdown

Concisely describe the change

This PR resolves runtime console errors across the podman.io website and broken relative documentation paths.

Key Changes:

  • Resolve DOM Nesting Warnings: Changed parent <p> wrappers to <div> elements in CustomCard.tsx and NotFound.tsx to fix invalid nesting violations (e.g. block-level Markdown elements/divs rendering inside paragraph tags).

  • Fix Unique Key Warning: Added key={idx} to the options.map iteration block inside the CommunityMeetingsCardGrid dropdown renderer to satisfy React list uniqueness requirements.

  • Fix Relative Docs Paths: Corrected broken relative documentation links in docs/index.md pointing to docs/installation and docs/checkpoint. They now correctly resolve relative to /docs (i.e. ./installation and ./checkpoint).

Before fix

Screenshot 2026-08-03 213329 Screenshot 2026-08-03 213318

After fix

image

Checklist

Ensure you have completed the following checklist for your pull request to be reviewed:

  • Certify you wrote the patch or otherwise have the right to pass it on as an open-source patch by signing all
    commits. (git commit -s). (If needed, use git commit -s --amend). The author email must match
    the sign-off email address. See CONTRIBUTING.md
    for more information.
  • Referenced issues using Fixes: #00000 in commit message (if applicable)
  • PR description, commit message, and GitHub comments are human-written, per LLM Policy

Signed-off-by: prakharsingh-74 <prakharsingh7014@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant