Skip to content

Commit 945ef73

Browse files
committed
app: don't calculate scrollTop anymore
Needs more logic to make it work as expected
1 parent 3e8dfa6 commit 945ef73

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

app.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,9 +99,13 @@ function App(el, currentWindow) {
9999
if (this.activeModel.ele) {
100100
const ele = document.querySelector(this.activeModel.ele)
101101
if (ele) {
102-
if (ele.scrollHeight - ele.clientHeight <= ele.scrollTop + 1) {
103-
ele.scrollTop = ele.scrollHeight
104-
}
102+
// TODO(evanlucas) Fix this so it auto scrolls only
103+
// if we are not already scrolling
104+
//
105+
// if (ele.scrollHeight - ele.clientHeight <= ele.scrollTop + 1) {
106+
// ele.scrollTop = ele.scrollHeight
107+
// }
108+
ele.scrollTop = 0
105109
}
106110
} else if (this.activeModel instanceof Settings) {
107111
const ele = document.querySelector('.settings-container')

0 commit comments

Comments
 (0)