Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed the issue of home page not directing in Leaderboard page. #97

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Crimson-Typhoon-147
Copy link
Contributor

Fixed the incorrect redirection of the "HOME" button on the leaderboard page.
Updated the href in the leaderboard HTML file to point to the main index.html located outside all folders by setting the path to ../../index.html.
This ensures that clicking the HOME button will now correctly navigate to the main homepage of the site.

main/index.html Show resolved Hide resolved
@Vinay-Khanagavi Vinay-Khanagavi added the Changes required Pull request is not good enough label Oct 21, 2024
@B1N4RY-P4R45173
Copy link
Collaborator

@Crimson-Typhoon-147 I am talking about the index.html in main. Sorry for late response I have away for a few days

@Crimson-Typhoon-147
Copy link
Contributor Author

Okay! Thank you for the clarification. making it.

@Crimson-Typhoon-147
Copy link
Contributor Author

Crimson-Typhoon-147 commented Oct 25, 2024

@B1N4RY-P4R45173 Just need one more thing to confirm I guess the index.html has been renamed into badges.html which redirects to the same page but there has been a css file badges.css and i don't think that it has anything to do with the badges.html file as it opens totally a different page.

so as @Vinay-Khanagavi had also assigned me the issue of desgning a leaderboard page I think it'll be better if I make changes in the badges.html file and rename it into leaderboard page and add the features.

Please review it and let me know what to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes required Pull request is not good enough
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants