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

Dashboard page updated #430

Closed
wants to merge 2 commits into from
Closed

Conversation

Nidhi-2408
Copy link

Description :

This was a change necessary for improved layout of the user.

Changes Made :
I improved the layout and styling of the user page

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you @Nidhi-2408 for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better

Copy link
Member

@Harshdev098 Harshdev098 left a comment

Choose a reason for hiding this comment

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

There are many changes required to be modified:

  • migrate the styling of dashboard to its css file
  • Also you have removed the li elements like logout and the view result is not working after modifications.
  • the home element should be linked to the main page ie. after login pages
  • Also edit the Edit profile button
  • The language translator is not showing anywhere

image

Also its good layout but i think you can improve it more 👍

@Nidhi-2408
Copy link
Author

@Harshdev098 Will work on it within two days

@Harshdev098
Copy link
Member

Okay no problem, take your time 👍

@Nidhi-2408 Nidhi-2408 closed this Jan 4, 2025
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.

2 participants