scratchfoundation/scratch-www
1
0
Fork 0
mirror of https://github.com/scratchfoundation/scratch-www.git synced 2025-02-17 00:21:20 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

better handling of addtostudio modal scrollbar; attempting to debug its setstate calls

Browse source
This commit is contained in:
Ben Wheeler 2018-06-19 10:47:17 -04:00
parent 16190f1147
commit b18ddeedaf
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/components/modal/addtostudio/modal.jsx
View file

@ -64,6 +64,7 @@ class AddToStudioModal extends React.Component {
}
componentWillReceiveProps(nextProps) {
debugger;
this.updateOnOrDirty(nextProps.projectStudios, nextProps.myStudios);
}
@ -140,6 +141,7 @@ class AddToStudioModal extends React.Component {
// onOrDirty should start with a clean slate
// NOTE: this doesn't seem to be working
this.setState({onOrDirty: {}});
debugger;
this.props.onAddToStudio(studiosToAdd, studiosToDelete, err => {
if (err) log.error(err);
this.setState({

6
src/components/modal/addtostudio/modal.scss
View file

@ -40,10 +40,8 @@
.studio-list-outer-scrollbox {
background-color: $ui-blue-10percent-solid;
padding-top: 10px;
}
.studio-list-inner-scrollbox {
height: 15rem;
overflow: scroll;
@ -60,6 +58,8 @@
border-radius: 5px;
}
.studio-list-inner-scrollbox::-webkit-scrollbar-track {
margin-top: 10px;
margin-bottom: 10px;
}
.studio-list-container {
@ -68,6 +68,8 @@
justify-content: flex-start;
flex-flow: row wrap;
padding: 1rem 3rem;
min-height: 30rem;
}
/* NOTE: force scrolling: add to above:
min-height: 30rem;