Changeset 142745 in webkit
- Timestamp:
- Feb 13, 2013 7:25:00 AM (11 years ago)
- Location:
- trunk/Source/WebCore
- Files:
-
- 4 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r142743 r142745 1 2013-02-13 Vladislav Kaznacheev <kaznacheev@chromium.org> 2 3 Web Inspector: Fixed colorpicker editing and scrolling. 4 https://bugs.webkit.org/show_bug.cgi?id=109434. 5 6 Reviewed by Alexander Pavlov. 7 8 The color picker scrolling logic relied on the fixed DOM structure which changed with the introduction of 9 SidebarPaneStack (https://bugs.webkit.org/show_bug.cgi?id=108183). 10 Added a special CSS class to mark the scroll target. 11 12 No new tests. 13 14 * inspector/front-end/SplitView.js: 15 (WebInspector.SplitView): 16 * inspector/front-end/StylesSidebarPane.js: 17 (WebInspector.StylePropertyTreeElement.prototype.updateTitle.): 18 * inspector/front-end/TabbedPane.js: 19 (WebInspector.TabbedPane): 20 1 21 2013-02-13 Andrey Lushnikov <lushnikov@chromium.org> 2 22 -
trunk/Source/WebCore/inspector/front-end/SplitView.js
r142738 r142745 43 43 this.element.className = "split-view"; 44 44 45 this._firstElement = this.element.createChild("div", "split-view-contents"); 46 this._firstElement.addStyleClass("split-view-contents-first"); 47 48 this._secondElement = this.element.createChild("div", "split-view-contents"); 49 this._secondElement.addStyleClass("split-view-contents-second"); 45 this._firstElement = this.element.createChild("div", "split-view-contents scroll-target split-view-contents-first"); 46 this._secondElement = this.element.createChild("div", "split-view-contents scroll-target split-view-contents-second"); 50 47 51 48 this._resizerElement = this.element.createChild("div", "split-view-resizer"); -
trunk/Source/WebCore/inspector/front-end/StylesSidebarPane.js
r142440 r142745 1772 1772 colorSwatch.element.addEventListener("click", swatchClick, false); 1773 1773 1774 var scrollerElement = hasSpectrum ? self._parentPane._computedStylePane.element.parentElement : null;1774 var scrollerElement; 1775 1775 1776 1776 function spectrumChanged(e) … … 1786 1786 function spectrumHidden(event) 1787 1787 { 1788 scrollerElement.removeEventListener("scroll", repositionSpectrum, false); 1788 if (scrollerElement) 1789 scrollerElement.removeEventListener("scroll", repositionSpectrum, false); 1789 1790 var commitEdit = event.data; 1790 1791 var propertyText = !commitEdit && self.originalPropertyText ? self.originalPropertyText : (nameElement.textContent + ": " + valueElement.textContent); … … 1818 1819 spectrumHelper.addEventListener(WebInspector.SpectrumPopupHelper.Events.Hidden, spectrumHidden); 1819 1820 1820 scrollerElement.addEventListener("scroll", repositionSpectrum, false); 1821 scrollerElement = colorSwatch.element.enclosingNodeOrSelfWithClass("scroll-target"); 1822 if (scrollerElement) 1823 scrollerElement.addEventListener("scroll", repositionSpectrum, false); 1824 else 1825 console.error("Unable to handle color picker scrolling"); 1821 1826 } 1822 1827 } -
trunk/Source/WebCore/inspector/front-end/TabbedPane.js
r138151 r142745 41 41 this._headerContentsElement = this._headerElement.createChild("div", "tabbed-pane-header-contents"); 42 42 this._tabsElement = this._headerContentsElement.createChild("div", "tabbed-pane-header-tabs"); 43 this._contentElement = this.element.createChild("div", "tabbed-pane-content ");43 this._contentElement = this.element.createChild("div", "tabbed-pane-content scroll-target"); 44 44 this._tabs = []; 45 45 this._tabsHistory = [];
Note: See TracChangeset
for help on using the changeset viewer.