scratchfoundation/scratch-www
1
0
Fork 0
mirror of https://github.com/scratchfoundation/scratch-www.git synced 2024-11-26 17:16:11 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

Merge pull request #3140 from benjiwheeler/margin-collapse-use-padding-instead

Browse source 
use padding to avoid margin collapse; drop approach that used `overflow: auto`
This commit is contained in:
Benjamin Wheeler 2019-07-16 15:29:17 -04:00 committed by GitHub
commit 88b3ea6698
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 10 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/components/join-flow/join-flow-step.scss
View file

@ -9,6 +9,8 @@
.join-flow-inner-content {
box-shadow: none;
width: calc(100% - 5.875rem);
margin: 2.3125rem auto 2.5rem;
/* must use padding for top, rather than margin, because margins will collapse */
margin: 0 auto;
padding: 2.3125rem 0 2.5rem;
font-size: .875rem;
}

2
src/components/modal/base/modal-inner-content.jsx
View file

@ -8,7 +8,6 @@ const ModalInnerContent = ({
children,
className
}) => (
<div className="modal-inner-clear">
<div
className={classNames(
'modal-inner-content',
@ -17,7 +16,6 @@ const ModalInnerContent = ({
>
{children}
</div>
</div>
);
ModalInnerContent.propTypes = {

9
src/components/modal/base/modal-inner-content.scss
View file

@ -1,15 +1,6 @@
@import "../../../colors";
@import "../../../frameless";
/*
necessary to prevent modal-inner-content's margins from
unexpectedly collapsing with a parent
*/
.modal-inner-clear {
overflow: auto;
height: 100%;
}
.modal-inner-content {
box-sizing: border-box;
display: flex;