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

Change logout button to User avatar with dropdown on hover #69 #71

Merged
merged 4 commits into from
Feb 6, 2024

Conversation

mohitparmar1
Copy link
Contributor

@mohitparmar1 mohitparmar1 commented Jan 28, 2024

Pull Request

add swoc label
Related Issues:
Change logout button to User avatar with dropdown on hover #69

Fixes #69

Description:
[Provide a brief description of your changes.]
once user login its only shows dummy profile image , onclick it will shows logout , profile ,etc,...

Note: Do [x] to select the below checkbox

Checklist:

  • I have tested my changes.
  • My code follows the project's coding standards.
  • I have updated the documentation (if applicable).

Screenshots:
[If applicable, include screenshots to help reviewers understand your changes.]

Additional information

Please provide any other i

Meal.Matters.-.Google.Chrome.2024-01-28.22-55-16.mp4

information that you think may be relevant to the pull request.

Copy link

@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.

Great job, @mohitparmar1! 🎉 Thank you for submitting your pull request for MealMatters. Our dedicated team will review it diligently.Your contribution is valuable and we appreciate your efforts to improve our project.

@AnupK1234
Copy link
Owner

@mohitparmar1
Isn’t the issue assigned to you? First get the issue assigned then create a PR

@mohitparmar1
Copy link
Contributor Author

@mohitparmar1 Isn’t the issue assigned to you? First get the issue assigned then create a PR

sorry , but I think no one was woking on that issue ,that's why I raised pr

@AnupK1234
Copy link
Owner

@mohitparmar1 Isn’t the issue assigned to you? First get the issue assigned then create a PR

sorry , but I think no one was woking on that issue ,that's why I raised pr

If you would like to work on that issue do let me know in the issue comment section. I will assign it.

Copy link
Owner

@AnupK1234 AnupK1234 left a comment

Choose a reason for hiding this comment

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

1] Make routing of Profile to "/dashboard"
2] On hover the dropdown should be visible and when the mouse gets back it should close the dropdown.

@AnupK1234
Copy link
Owner

@mohitparmar1
Are you done with this ?

Copy link
Owner

@AnupK1234 AnupK1234 left a comment

Choose a reason for hiding this comment

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

The dropdown isn't visible on click and nor onhover.

@mohitparmar1
Copy link
Contributor Author

need some time to complete

@AnupK1234 AnupK1234 added Accepted PR accepted and removed title needs formatting labels Feb 6, 2024
@AnupK1234 AnupK1234 merged commit 7874e11 into AnupK1234:main Feb 6, 2024
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: Change logout button to User avatar with dropdown on hover
2 participants