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

Create Bootstrap documentation & styles for functioning mobile nav #104

Merged
merged 98 commits into from Sep 25, 2020

Conversation

trackleft
Copy link
Member

@trackleft trackleft commented Jul 24, 2020

Closes #90

joeparsons and others added 30 commits June 23, 2020 13:57
mmunro-ltrr
mmunro-ltrr previously approved these changes Sep 9, 2020
AZ Quickstart automation moved this from In progress to Review in progress Sep 9, 2020
@akslay
Copy link
Contributor

akslay commented Sep 11, 2020

It looks like there's a weird overlap happening with the dropdowns (this is after refreshing the page at the mobile screen size).
Screen Shot 2020-09-11 at 11 30 04 AM

It looks like the problem is coming from JS somewhere. In the inspector, styles are being added dynamically. And if you uncheck all those styles, the dropdowns appear correctly.
Screen Shot 2020-09-11 at 11 35 38 AM

@@ -109,6 +109,11 @@
font-family: $font-family-sans-serif;
font-weight: 500;
}
// >>> Material design icons.
.material-icons {
font-size: inherit;
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like this style is preventing utility classes (like .display-3) from working as expected; it's not allowing the class to change the font size.

@joeparsons joeparsons changed the base branch from master to main September 16, 2020 17:34
@mmunro-ltrr mmunro-ltrr self-requested a review September 25, 2020 17:58
Copy link
Member

@mmunro-ltrr mmunro-ltrr left a comment

Choose a reason for hiding this comment

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

Previous approval was in the distant past, so wasn't really relevant.

AZ Quickstart automation moved this from Review in progress to Reviewer approved Sep 25, 2020
@trackleft trackleft changed the title Update feature/93 to include reference js plugin based on collapse.js and adding version var Create Bootstrap documentation & styles for functioning mobile nav Sep 25, 2020
@trackleft trackleft merged commit d645774 into main Sep 25, 2020
AZ Quickstart automation moved this from Reviewer approved to Done Sep 25, 2020
@kevcooper kevcooper deleted the feature/93-chris branch September 25, 2020 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
AZ Quickstart
  
Done
Development

Successfully merging this pull request may close these issues.

Create Bootstrap documentation & styles for functioning mobile nav
5 participants