specollective / insights-agent-web-app

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

48 web app UI styling for download page #60

Closed najwanan closed 1 year ago

najwanan commented 1 year ago

Ticket Description <-- Copy and paste or write the tickets description here--> UI styling for download page

Screenshot 2022-12-03 at 9 48 41 AM Screenshot 2022-12-03 at 9 49 14 AM

Description of Changes <-- Brief description of the changes you made, files you touched, etc -->

Before and After for UI Updates <-- You should have 2 images minimum, for desktop, 4 for mobile and desktop combined -->

Before:

After:

Screenshot 2022-12-03 at 9 51 10 AM Screenshot 2022-12-03 at 9 50 37 AM

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?

najwanan commented 1 year ago

Thanks Daniel!

Did some reading up on best practices for classNames, and sr-only class. Most of it was pretty straightforward, just wasn't sure if I interpreted the issue with h1 and sr-only correctly. I'll touch base in the morning, other than that this should be good to go!

najwanan commented 1 year ago

@dashcraft This one has been update with the h1 tags and sr-only. Should be good to merge now? 🤞🏽

najwanan commented 1 year ago

Thanks, pulled it down and tested it. Looks good.