Skip to content

Commit

Permalink
Remove some obsolete logic
Browse files Browse the repository at this point in the history
Now that the scroller div has its own vertical scrollbar, we no longer
need to explicitly update the scrollbar during updateDisplay.
  • Loading branch information
marijnh committed Aug 17, 2012
1 parent f44ac12 commit dde025c
Showing 1 changed file with 1 addition and 10 deletions.
11 changes: 1 addition & 10 deletions lib/codemirror.js
Expand Up @@ -1100,16 +1100,7 @@ var CodeMirror = (function() {
return heightChanged;
}

if (options.lineWrapping) {
checkHeights();
var scrollHeight = needsScrollbar();
var shouldHaveScrollbar = scrollHeight ? "block" : "none";
if (scrollbar.style.display != shouldHaveScrollbar) {
scrollbar.style.display = shouldHaveScrollbar;
if (scrollHeight) scrollbarInner.style.height = scrollHeight + "px";
checkHeights();
}
}
if (options.lineWrapping) checkHeights();

gutter.style.display = gutterDisplay;
if (different || gutterDirty) {
Expand Down

0 comments on commit dde025c

Please sign in to comment.