-
Notifications
You must be signed in to change notification settings - Fork 395
Conversation
font-size: 12px; | ||
} | ||
|
||
.footer-list { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.docs-footer-list
width: 8.33%; | ||
} | ||
|
||
.footer-cc { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.docs-footer-copyright
padding: 8px; | ||
} | ||
|
||
.docs-angular-logo { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.docs-footer-angular-logo
</div> | ||
|
||
<div class="footer-links"> | ||
<ul class="footer-links"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is having two elements with the same class intentional?
justify-content: flex-end; | ||
} | ||
|
||
footer ul { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can these styles use the css classes instead? There's already a class on the ul
flex-direction: column; | ||
} | ||
.footer-logo { | ||
width: 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
display: none
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Added Angular logo and link to angular.io in footer to match microsite templates