Skip to content

Update modal footer links to balance vertical spacing with flex

Eric Eastwood requested to merge bugfix/modal-button-link-alignment-flex into master

Update modal footer links to balance vertical spacing with flex. If we instead want to use margin, do not merge this, instead https://github.com/gitterHQ/ui-components/pull/5. note: we can achieve either vertical space effect with both flex or margin, but flex is just easier in alignment regards.

Fixes troupe/gitter-webapp#702

![](https://i.imgur.com/7BBnatw.png)

I saw the following comment in the CSS while updating. This PR will work fine in IE11 and Edge which is the latest. I just had to add justify-content: flex-start; to the flex parent as well.

/* Flexbox layout break in IE.
 * Migrating to floats solved the problem but you don't get nice vertical centring
 * therefore we need to add a margin-top to text links ... ugh
 * jp 5/11/15
 * */

source

Merge request reports