Skip to content

Fix invalid HTML structure (unclosed <main>, orphan </footer>) on index, contact, shop pages#162

Merged
DavidKRK merged 2 commits into
gh-pagesfrom
copilot/fix-page-issues
Apr 7, 2026
Merged

Fix invalid HTML structure (unclosed <main>, orphan </footer>) on index, contact, shop pages#162
DavidKRK merged 2 commits into
gh-pagesfrom
copilot/fix-page-issues

Conversation

Copilot AI commented Apr 7, 2026

Copy link
Copy Markdown
Contributor
  • Identify the bug: <main> tag not closed (no </main>) and stray </footer> without opening <footer> in index.html, contact.html, shop.html
  • Fix index.html: close </main> after player section, wrap footer content in <footer>
  • Fix contact.html: close </main> after contact forms, wrap footer content (including social icons) in <footer>
  • Fix shop.html: close </main> after shop content, wrap footer content in <footer>
  • Move social icon links from inside <main> into <footer> in contact.html (reviewer feedback)

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes malformed HTML on key pages by properly closing <main> and introducing a matching <footer> wrapper to eliminate orphan </footer> tags and restore semantic structure.

Changes:

  • Add missing </main> closing tags on index.html, contact.html, and shop.html.
  • Introduce <footer>…</footer> blocks after main content to ensure valid footer structure.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
index.html Closes <main> and wraps footer content in a proper <footer> element.
contact.html Closes <main> and adds a <footer> element (but footer contents differ from PR description).
shop.html Closes <main> and wraps footer content in a proper <footer> element.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread contact.html Outdated
Copilot AI requested a review from DavidKRK April 7, 2026 17:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@DavidKRK DavidKRK marked this pull request as ready for review April 7, 2026 17:23
@DavidKRK DavidKRK merged commit 5ab541f into gh-pages Apr 7, 2026
13 checks passed
@DavidKRK

DavidKRK commented Apr 8, 2026

Copy link
Copy Markdown
Owner

@copilot apply changes based on the comments in this thread

DavidKRK added a commit that referenced this pull request Apr 8, 2026
… index, contact, shop pages (#162) (#163)

* Fix missing main/footer HTML structure in index, contact, shop pages

Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/22a9b0b3-0b65-40f6-8d32-31b6fb1f1905



* Move social icons into footer in contact.html

Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/73fb46a3-65a6-497f-90fe-72daccefd627



---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
DavidKRK added a commit that referenced this pull request Apr 8, 2026
… index, contact, shop pages (#162) (#163) (#164)

* Fix missing main/footer HTML structure in index, contact, shop pages

Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/22a9b0b3-0b65-40f6-8d32-31b6fb1f1905



* Move social icons into footer in contact.html

Agent-Logs-Url: https://github.com/DavidKRK/DavidKRK.github.io/sessions/73fb46a3-65a6-497f-90fe-72daccefd627



---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.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.

3 participants