specollective / insights-agent-web-app

Web client for the Insights Agent application.
2 stars 1 forks source link

Issue #57: Mobile Nav #59

Closed YolineB closed 1 year ago

YolineB commented 1 year ago

Ticket Description

Fixes #57 Navbar for mobile as list in Figma for mobile

image

Note: Navbar logo needs to be updated with correct styling as mentioned in issue #58

Description of Changes

Remove logo image, add logo text, correct font weight for language selected, limit nav to display only on mobile

Before and After for UI Updates

Before:

Desktop:

image

Mobile:

image

After:

Desktop:

image

Mobile:

image

For PR Reviewer Does this file change the yarn.lock, package.json or package-lock.json file? If so, why? If this pr contains mobile and desktop changes, did you test on IphoneXr and desktop views? Does this file match the related tickets linked figma file, or does it pass the visual smell test? If this file contains javascript, does the javascript pass the smell test? If you don't feel super confident in your review, did you assign someone more senior to double check?