Skip to content

fixed home in footer link of index page#298

Open
zyzpaprika wants to merge 1 commit intoopensource-society:mainfrom
zyzpaprika:main
Open

fixed home in footer link of index page#298
zyzpaprika wants to merge 1 commit intoopensource-society:mainfrom
zyzpaprika:main

Conversation

@zyzpaprika
Copy link

@zyzpaprika zyzpaprika commented Jul 28, 2025

invalid ID attribute in the footer of the index page, now fixed!

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Greptile Summary

This PR fixes a navigation issue in the index.html file where the home link in the footer was not functioning correctly. The problem was that the hero section had an invalid ID attribute containing a space (id="homeSection top") which prevented proper anchor link navigation. The change corrects this by updating the ID to id="homeSection" (line 148), ensuring that when users click the "Home" link in the footer, it properly scrolls to the hero section.

This is a straightforward bug fix that addresses a basic user experience issue with the site's navigation. The footer contains anchor links to different sections of the page, and this fix ensures the home link works as intended alongside the other footer navigation links.

Confidence score: 1/5

  • This PR addresses the specific navigation issue but the index.html file contains critical structural problems that make it unsafe to merge
  • The file has severe HTML validation issues including duplicate <main> tags, duplicate navigation sections, and improper nesting that would break the page functionality
  • Files needing attention: index.html requires significant structural cleanup before this fix can be safely merged

1 file reviewed, no comments

Edit Code Review Bot Settings | Greptile

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