Skip to content

fix duplicate home buttons#378

Open
saurabhhhcodes wants to merge 1 commit into
Kuldeeep18:mainfrom
saurabhhhcodes:fix/remove-duplicate-home-buttons-320
Open

fix duplicate home buttons#378
saurabhhhcodes wants to merge 1 commit into
Kuldeeep18:mainfrom
saurabhhhcodes:fix/remove-duplicate-home-buttons-320

Conversation

@saurabhhhcodes

@saurabhhhcodes saurabhhhcodes commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Removed the duplicate "Back to Home" button from frontend/login.html.
  • Removed the duplicate "Back to Home" button from frontend/register.html.

Why

  • Both auth pages were rendering two identical home links, which was redundant and cluttered the UI.

Validation

  • rg -n "Back to Home" frontend/login.html frontend/register.html
  • git diff --check

Closes #320

Summary by CodeRabbit

  • Style
    • Reorganized the "Back to Home" button structure on the login page for improved layout consistency.
    • Repositioned the "Back to Home" button on the registration page for better visual flow.

@coderabbitai

coderabbitai Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 37a41289-7dc3-4db2-9e4b-64f6405e3c8a

📥 Commits

Reviewing files that changed from the base of the PR and between 0474fe9 and d040927.

📒 Files selected for processing (2)
  • frontend/login.html
  • frontend/register.html
💤 Files with no reviewable changes (2)
  • frontend/login.html
  • frontend/register.html

📝 Walkthrough

Walkthrough

Removes duplicate "Back to Home" button markup from login.html and register.html. In login.html, the link's surrounding tag structure is corrected. In register.html, the earlier occurrence of the button near the top of the card is deleted, leaving only the instance inside the mb-3 container.

Changes

Duplicate "Back to Home" Button Removal

Layer / File(s) Summary
DOM structure fix in login and register pages
frontend/login.html, frontend/register.html
Removes 7 lines of redundant markup from each file: login.html fixes the surrounding tag structure of the "Back to Home" link, and register.html deletes the earlier duplicate occurrence of the button near the start of the card content.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

frontend, type:bug, level:beginner

🐇 Two buttons where one should be,
A duplicate hiding in the HTML tree.
I hopped through the divs with a twitch of my nose,
Snipped out the extra — now the page tidily flows!
One "Back to Home" is all that we need. 🏠

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix duplicate home buttons' clearly summarizes the main change: removing redundant home link elements from authentication pages.
Linked Issues check ✅ Passed The PR successfully addresses all acceptance criteria from issue #320: duplicate buttons removed from both login.html and register.html with no other changes.
Out of Scope Changes check ✅ Passed All changes are scoped to removing duplicate 'Back to Home' buttons from the two specified files with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@Kuldeeep18

Copy link
Copy Markdown
Owner

Hi @saurabhhhcodes 👋

LeadOrbit Bot here 🤖

A repository star ⭐ is required for PR review and merge.

We noticed that you haven't starred the repository yet. Please star the repo and reply once completed so we can continue with the review process.

Thanks for contributing to LeadOrbit! 🚀

@Kuldeeep18

Copy link
Copy Markdown
Owner

Hi @saurabhhhcodes 👋

LeadOrbit Bot here 🤖

We noticed you've opened a Pull Request but haven't starred the repository yet.

Starring the repository is mandatory for PR review and merge.

Please:

  1. Star the repository.
  2. Reply to this comment with "Done".

Once you've done that, the bot will continue processing your PR.

Note: PRs from contributors who haven't starred the repository will remain pending until this requirement is completed.

Thanks for contributing to LeadOrbit! 🚀

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

2 participants