[Webkit-unassigned] [Bug 46738] Web Inspector: add event listener breakpoints sidebar pane

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Oct 5 06:55:14 PDT 2010


https://bugs.webkit.org/show_bug.cgi?id=46738





--- Comment #8 from Pavel Podivilov <podivilov at chromium.org>  2010-10-05 06:55:13 PST ---
> NIT: Extra newline up top.
> 
> I think these keys should be localized, since they are the title of TreeElement's displayed to the user.
> Would be easy to solve with along the lines of: (and adding the strings.
> 
>   var categoryTreeElement = new TreeElement(WebInspector.UIString(category));
> 
done

> 
> > WebCore/inspector/front-end/EventListenersSidebarPane.js:186
> > +    this.propertiesElement.className = "event-properties properties-tree source-code"; /* Changed from "properties" */
> 
> How does this relate to your other changes? How does this affect the Event
> Listeners sidebar pane in the Element's panel?
> 
> 
> > WebCore/inspector/front-end/PropertiesSection.js:35
> > +    this.propertiesElement.className = "properties properties-tree source-code";
> 
> Same comment. How does this affect the properties pane in the Elements panel?

It doesn't affect the panels, it's just a refactoring of css.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list