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

Feature/tas 12 nav bar #25

Closed
wants to merge 4 commits into from
Closed

Feature/tas 12 nav bar #25

wants to merge 4 commits into from

Conversation

rachelsitoh
Copy link
Contributor

Pull Request

Description

Nav bar for desktop and mobile

Related Issues

[Reference any related issues that this pull request addresses or resolves.]

Checklist

Please review and check the following before submitting your pull request:

  • I have read the CONTRIBUTING.md document.
  • My code follows the code style of this project.
  • I have written tests for my changes (if applicable).
  • All new and existing tests passed.
  • I have updated the documentation accordingly (if applicable).
  • I have added comments to my code, particularly in areas that may be confusing.

Desktop

Screenshot 2024-02-13 at 9 37 45 PM

Mobile

Screenshot 2024-02-13 at 9 40 14 PM
Screenshot 2024-02-13 at 9 40 07 PM

Additional Information

I assumed clicking on the profile logo will make the 'box' containing profile and sign out appear, and clicking it again will remove the 'box'. (For desktop, clicking on the white areas outside the header will make the 'box' disappear). lmk if any changes are needed. Also, I assume that clicking on the fintech logo will route to /

@rachelsitoh rachelsitoh requested a review from gnimnix February 13, 2024 14:54
@gnimnix gnimnix closed this Feb 14, 2024
@gnimnix gnimnix deleted the feature/TAS-12-nav-bar branch February 14, 2024 15:31
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