Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Fix home button alignment
Browse files Browse the repository at this point in the history
Auditors: @bsclifton

Fix #4513
  • Loading branch information
bbondy committed Oct 4, 2016
1 parent e2037f6 commit 492daf8
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions less/button.less
Original file line number Diff line number Diff line change
Expand Up @@ -157,4 +157,9 @@ span.browserButton,
color: white;
}
}

&.homeButton {
margin-bottom: auto;
margin-top: auto;
}
}

3 comments on commit 492daf8

@bsclifton
Copy link
Member

@bsclifton bsclifton commented on 492daf8 Oct 4, 2016

Choose a reason for hiding this comment

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

LGTM 😄 ++

@bbondy
Copy link
Member Author

@bbondy bbondy commented on 492daf8 Oct 5, 2016

Choose a reason for hiding this comment

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

this was only on a branch, and was just to try it on Windows. I couldn't repro on mac. I did reproduce it on Windows and the above didn't fix though so I won't merge this branch.

@bbondy
Copy link
Member Author

@bbondy bbondy commented on 492daf8 Oct 5, 2016

Choose a reason for hiding this comment

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

the proper fix is just to move the home button inside the startButtons class

Please sign in to comment.