Skip to content

fix: remove duplicate Back to Home buttons#632

Open
KratikaShukla07 wants to merge 1 commit into
Kuldeeep18:mainfrom
KratikaShukla07:fix/lo-072-remove-duplicate-back-home
Open

fix: remove duplicate Back to Home buttons#632
KratikaShukla07 wants to merge 1 commit into
Kuldeeep18:mainfrom
KratikaShukla07:fix/lo-072-remove-duplicate-back-home

Conversation

@KratikaShukla07

@KratikaShukla07 KratikaShukla07 commented Jul 8, 2026

Copy link
Copy Markdown

🔗 Related Issue

Closes #320

📝 Summary of Changes

  • Removed the duplicate "Back to Home" button from frontend/login.html
  • Removed the duplicate "Back to Home" button from frontend/register.html
  • Ensured only one button is displayed on each page
  • Preserved the existing layout and navigation functionality

🧪 Testing

  • Verified Login page displays a single "Back to Home" button
  • Verified Register page displays a single "Back to Home" button
  • Confirmed the button correctly navigates back to the home page

✅ Checklist

  • No merge conflicts
  • Changes tested locally
  • Follows project guidelines
  • Related issue linked

Summary by CodeRabbit

  • Bug Fixes
    • Cleaned up the login page layout by removing duplicate back-navigation markup and aligning the “Back to Home” button with the intended spacing.
    • Simplified the registration page layout by removing redundant back button container markup and improving spacing before the page header.

@KratikaShukla07

Copy link
Copy Markdown
Author

Hi @Kuldeeep18,

I have completed the requested changes for this issue.

The duplicate "Back to Home" buttons have been removed from both the Login and Register pages while preserving the existing functionality.

Kindly review the PR when you have time.

Thank you!

@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Removed duplicated "Back to Home" button markup from login.html and register.html, leaving a single back-link block within the intended container before the page header in each file. No form fields or JavaScript logic were changed.

Changes

Duplicate button cleanup

Layer / File(s) Summary
Remove duplicate back button markup
frontend/login.html, frontend/register.html
Removes the redundant wrapper/container HTML for the "Back to Home" button, keeping a single back-link block before the page header.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested labels: frontend, type:bug, level:beginner

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The changes remove the extra button markup from both login.html and register.html as requested in issue #320.
Out of Scope Changes check ✅ Passed The PR stays focused on the two targeted HTML files and only cleans up duplicate button markup.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: removing duplicate Back to Home buttons from the login and register pages.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

LO-072 [Easy - Bug]: Fix Duplicate 'Back to Home' Buttons on Login and Register Pages

1 participant