Skip to content

Modification in header of the project for better UI#302

Open
Akanksha6209127 wants to merge 23 commits into
Amarjha01:masterfrom
Akanksha6209127:new
Open

Modification in header of the project for better UI#302
Akanksha6209127 wants to merge 23 commits into
Amarjha01:masterfrom
Akanksha6209127:new

Conversation

@Akanksha6209127

Copy link
Copy Markdown

I try to made cleaner version for header section, increase the size of left side icon, add merge all the components of header in hamburger menu for desktop. Sign In, Dark/Light toggle button and Get Started is not added in hamburger menu it is directly shown in header section. I also change the position of Moon icon in toggle button of Dark/Light theme. These changes made a clean and better UI and it is easy to use and no any components are overlapped.
These following images shows the changes in features of UI.
Before:-
2025-09-27
After:-
2025-09-27 (4)
2025-09-27 (2)
These Images shows the difference and changes. Please check the code.

…pand in header now because in expanded version the components overlap.

@Neha Neha left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

thank you for your PR.

review the comment please

const handleLogout = () => {
localStorage.removeItem('token');
localStorage.removeItem('userData');
localStorage.removeItem("token");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thank you.

if you want to do formatting fixes then please do it on whole project also better to create a editor rule file and add the rules there or add eslint config file. ESLint will do all formatting changes in one go.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Got it, thank you for the suggestion. I will configure ESLint rules for consistent formatting instead of fixing it file by file.

Comment thread client/src/components/Header.jsx Outdated
Comment thread client/src/components/Header.jsx Outdated
Comment thread client/src/components/Header.jsx Outdated
Comment thread client/src/components/Header.jsx Outdated
@CHHemant

CHHemant commented Oct 1, 2025

Copy link
Copy Markdown
Collaborator

@Akanksha6209127 This branch has conflicts that must be resolved

@Akanksha6209127

Copy link
Copy Markdown
Author

Please, check and merge the code. I try to made cleaner version for header section, increase the size of left side icon, No any components are overlapped, icon size increases, it looks clean now. I decrease the spaces between these buttons Sign In, Dark/Light toggle button and Get Started it looks better because it remove overlapping. I also change the position of Moon icon in toggle button of Dark/Light theme. These changes made a clean and better UI and it is easy to use and no any components are overlapped.
These following images shows the changes in features of UI.
After:-
2025-09-27
Before:-
2025-10-04 (1)

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.

3 participants