From 16abfcbe75dd8c7cc52fa9debf6bed548aa5b9ad Mon Sep 17 00:00:00 2001 From: Fabian Jakobs Date: Fri, 9 Dec 2011 16:49:27 +0100 Subject: [PATCH] Revert "add scrollbar before scroller to not cover it" This reverts commit decebfc552981390f83455579b92572b5903d47f. --- lib/ace/scrollbar.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/ace/scrollbar.js b/lib/ace/scrollbar.js index cc72e44b..64c418df 100644 --- a/lib/ace/scrollbar.js +++ b/lib/ace/scrollbar.js @@ -50,17 +50,16 @@ var ScrollBar = function(parent) { this.inner = dom.createElement("div"); this.element.appendChild(this.inner); + parent.appendChild(this.element); + // in OSX lion the scrollbars appear to have no width. In this case resize // the to show the scrollbar but still pretend that the scrollbar has a width // of 0px // in Firefox 6+ scrollbar is hidden if element has the same width as scrollbar // make element a little bit wider to retain scrollbar when page is zoomed - // and insert it before other siblings to not cover them this.width = dom.scrollbarWidth(parent.ownerDocument); this.element.style.width = (this.width || 15) + 5 + "px"; - parent.insertBefore(this.element, parent.firstChild); - event.addListener(this.element, "scroll", this.onScroll.bind(this)); };