Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

Fix spacing when app navigation isn't configured #583

Merged
merged 6 commits into from
Oct 18, 2017

Conversation

thevoiceofzeke
Copy link
Contributor

In this PR:

  • Added some CSS to maintain margins with or without side nav menu button
  • Removed a little unused CSS
  • Autohide menu tooltip so it doesn't linger on mobile

Contributor License Agreement adherence:

Copy link
Contributor

@davidmsibley davidmsibley left a comment

Choose a reason for hiding this comment

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

The adjacent selector seems like it'd be fragile. I can see myself breaking it really easily, but I don't have any actual experience to say don't use it.

@@ -67,6 +54,20 @@ portal-header {
.menu-toggle.md-button {
min-width: 48px;
margin: 0;

+ .title-link h1 {
Copy link
Contributor

Choose a reason for hiding this comment

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

❓ would a general sibling selector be more robust?

}

&.hide-gt-xs {
+ .title-link h1 {
Copy link
Contributor

Choose a reason for hiding this comment

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

❓ would a general sibling selector be more robust?

@thevoiceofzeke
Copy link
Contributor Author

@ChristianMurphy @davidmsibley Good catch :)

Copy link
Contributor

@ChristianMurphy ChristianMurphy left a comment

Choose a reason for hiding this comment

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

Thanks @thevoiceofzeke!

@vertein
Copy link
Contributor

vertein commented Oct 18, 2017

@thevoiceofzeke, I'm going to cut a release soon. This seems good to get in quick via merging?

@thevoiceofzeke thevoiceofzeke merged commit 2143eeb into uPortal-Attic:master Oct 18, 2017
@vertein vertein added this to the 6.1.0 milestone Oct 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants