Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

Fix sidenav not allowing flex children #737

Closed
wants to merge 1 commit into from

Conversation

epelc
Copy link
Contributor

@epelc epelc commented Nov 22, 2014

This fixes #734 which prevented you from aligning items to the bottom of a side nav because "flex" wouldn't work on child elements. The display:flex on .md-closed-add and .md-closed-remove also prevents you from seeing a pop from elements when you open/close the sidenav when you use display:flex on the open state.

@googlebot
Copy link

CLAs look good, thanks!

@ajoslin
Copy link
Contributor

ajoslin commented Nov 24, 2014

LGTM, thanks.

@ajoslin ajoslin closed this Nov 24, 2014
ajoslin pushed a commit that referenced this pull request Nov 24, 2014
@ajoslin
Copy link
Contributor

ajoslin commented Nov 24, 2014

Merged! Additionally added flex-direction: column; to the sidenav by default.

@epelc
Copy link
Contributor Author

epelc commented Nov 24, 2014

Cool that will get rid of an extra layout="column" now too.

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.

Why does md-sidenav use display:block instead of display:flex
3 participants