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

AS #394 - Redesign navbar and add NASSR logo #400

Merged
merged 1 commit into from
Mar 18, 2022
Merged

Conversation

camdendotlol
Copy link

@camdendotlol camdendotlol commented Mar 18, 2022

What this PR does

  1. Adds a link to NASSR in the top bar
  2. Redesigns the navbar to prevent it from covering up the page content on smaller screens. This was an existing issue exacerbated by the addition of NASSR to the navbar.

For the navbar redesign, I stuck to the Bootstrap 3.1.1 classes that the rest of AS uses, as well as a few custom styles.

Before

Screen.Recording.2022-03-18.at.12.14.12.PM.mov

After

Screen.Recording.2022-03-18.at.12.10.26.PM.mov

How to test

Visit as many parts of the site as possible on various screen sizes and make sure the navbar...

  1. Never expands to two lines
  2. Always shows "COVE Studio | Tenant Name" on the left
  3. Always offers login and other menu options, either on the navbar or in a dropdown inside it

I would give some links to different sections of the site, but I'm not familiar with all that's on there and I don't have access to the document view.

@jamiefolsom jamiefolsom temporarily deployed to cove-studio-cm-nassr-lo-6tm7cv March 18, 2022 16:15 Inactive
add NASSR logo to all brand views

remove extra slash from NASSR URL

hide logos on small screens

compile CSS changes

use hamburger menu in navbar

center items in dropdown menu

make the brand text smaller on all displays

display logos in mobile dropdown menu

make login dropdown centered in mobile dropdown

hide logos on tablet-size screens

expand width to hide logos

update alabama brand with new navbar

update bavs brand with new navbar

update styles to center all forms in mobile dropdown

switch remaining tenants to new navbar
@jamiefolsom jamiefolsom temporarily deployed to cove-studio-cm-nassr-lo-m6fagl March 18, 2022 20:06 Inactive
@camdendotlol camdendotlol merged commit 987ffa5 into master Mar 18, 2022
@camdendotlol camdendotlol deleted the cm/nassr-logo branch March 18, 2022 20:12
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.

None yet

2 participants