Skip to content

Feat(auth): Add theme toggle and adjust Back button on Login & Signup - #240

Merged
MannuVilasara merged 5 commits into
braydenidzenga:mainfrom
KhanjarSingh:feat/auth-back-theme-1761844339
Oct 31, 2025
Merged

Feat(auth): Add theme toggle and adjust Back button on Login & Signup#240
MannuVilasara merged 5 commits into
braydenidzenga:mainfrom
KhanjarSingh:feat/auth-back-theme-1761844339

Conversation

@KhanjarSingh

Copy link
Copy Markdown
Contributor

📋 Description

This PR updates the Login and Signup pages with two key UI improvements:

  • Added a theme toggle button for switching between light and dark modes.
  • Adjusted the Back button placement for better alignment and UX.
  • Implemented a fallback route to "/" if no navigation history exists.

🔗 Related Issue

Fixes #233


🧩 Type of Change

  • 🐛 Bug fix
  • 🚀 New feature
  • 🧹 Code refactor
  • 📝 Documentation update
  • ✅ Test addition or update
  • ⚙️ Other (please describe):

🧪 How Has This Been Tested?

  • Tested manually on both desktop and mobile.
  • Verified theme toggle persists correctly between pages.
  • Checked Back button works in all navigation cases.

📸 Screenshots (if applicable)

White theme desktop:-
image

white theme mobile :-
image

black theme mobile :-
image

black theme desktop:-
image


🧠 Additional Context

This feature improves usability and accessibility for authentication pages.

@ikeshav26

Copy link
Copy Markdown
Contributor

@KhanjarSingh I pulled the PR and checked it !!...
everyhting looking good but i think We need soem change for large devices ie. you had added the back button to the top left of page but i am talking about form's top left

@KhanjarSingh

Copy link
Copy Markdown
Contributor Author

@ikeshav26 i will try that way it is like that across the pages so i applied that i will do it ur way and push u can check compare and merge i will just right away do it dont sleep :) be active 😆

@KhanjarSingh

Copy link
Copy Markdown
Contributor Author

@ikeshav26 i have completed the changes u can check

@ikeshav26

Copy link
Copy Markdown
Contributor

@ikeshav26 i have completed the changes u can check

@KhanjarSingh I will check within some time actually I am now on windows :)

@KhanjarSingh

Copy link
Copy Markdown
Contributor Author

@ikeshav26 i have completed the changes u can check

@KhanjarSingh I will check within some time actually I am now on windows :)

@ikeshav26 demn too many gadgets :) , np take your time do review and tell asap after u move on to mac or whtever so i can fix fast :)

@ikeshav26

Copy link
Copy Markdown
Contributor

@KhanjarSingh I think previous one is looking better.. What's your thoughts

@KhanjarSingh

KhanjarSingh commented Oct 30, 2025

Copy link
Copy Markdown
Contributor Author

@ikeshav26 That was good but then that issue persist who will see on such and extreme end if ther have big screens or monitor in mobile view it both sct th same now it's ur and @MannuVilasara desicion

@ikeshav26

Copy link
Copy Markdown
Contributor

@ikeshav26 That was good but then that issue persist who will see on such and extreme end if ther have big screens or monitor in mobile view it both sct th same now it's ur and @MannuVilasara desicion
@KhanjarSingh actually in your last commit when back button is on form's top left its not placed at top properly like see in screenshot fix it first. make it fixed for both login and signup page

Screenshot 2025-10-31 at 02-07-43 ZettaNote — Beautiful Markdown Notes So, at last i think its good to make the both icons properly placed on top and we will add this

@KhanjarSingh

Copy link
Copy Markdown
Contributor Author

@ikeshav26 on it Sir 🫡

@KhanjarSingh

Copy link
Copy Markdown
Contributor Author
image image @ikeshav26 is that fine or any more adjustment in any of the screen?

@ikeshav26

Copy link
Copy Markdown
Contributor

image image @ikeshav26 is that fine or any more adjustment in any of the screen?
Now seems to be perfect!!....
We'll merge this PR soon

@KhanjarSingh

Copy link
Copy Markdown
Contributor Author

@ikeshav26 have addressed the issues and pushed the changes u can review and merge.

@MannuVilasara
MannuVilasara merged commit 3168b2a into braydenidzenga:main Oct 31, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]Create a theme toggler button on the signup and sign-in pages

3 participants