Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
C
coopwire-hypothesis
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
孙灵跃 Leon Sun
coopwire-hypothesis
Commits
53cebeca
Commit
53cebeca
authored
Mar 29, 2017
by
Sean Roberts
Committed by
GitHub
Mar 29, 2017
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #321 from hypothesis/fix-scroll-to-new-annotation
Fix scrolling to new annotations
parents
449d4cfc
09086090
Changes
1
Show whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
1 addition
and
1 deletion
+1
-1
thread-list.js
src/sidebar/components/thread-list.js
+1
-1
No files found.
src/sidebar/components/thread-list.js
View file @
53cebeca
...
@@ -79,7 +79,7 @@ function ThreadListController($scope, VirtualThreadList) {
...
@@ -79,7 +79,7 @@ function ThreadListController($scope, VirtualThreadList) {
// Note: This assumes that the element occupies the entire height of the
// Note: This assumes that the element occupies the entire height of the
// containing document. It would be preferable if only the contents of the
// containing document. It would be preferable if only the contents of the
// <thread-list> itself scrolled.
// <thread-list> itself scrolled.
var
maxYOffset
=
document
.
body
.
client
Height
-
window
.
innerHeight
;
var
maxYOffset
=
document
.
body
.
scroll
Height
-
window
.
innerHeight
;
return
Math
.
min
(
maxYOffset
,
visibleThreads
.
yOffsetOf
(
id
));
return
Math
.
min
(
maxYOffset
,
visibleThreads
.
yOffsetOf
(
id
));
}
}
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment