Title: [174254] trunk/Source/WebInspectorUI
- Revision
- 174254
- Author
- [email protected]
- Date
- 2014-10-02 19:07:17 -0700 (Thu, 02 Oct 2014)
Log Message
Web Inspector: Cannot add a new CSS Rule to elements with 3 or more class names
https://bugs.webkit.org/show_bug.cgi?id=137375
Patch by Joseph Pecoraro <[email protected]> on 2014-10-02
Reviewed by Timothy Hatcher.
Correctly merge all class names with dots instead of just the first two.
* UserInterface/Models/DOMNode.js:
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/ChangeLog (174253 => 174254)
--- trunk/Source/WebInspectorUI/ChangeLog 2014-10-03 01:40:12 UTC (rev 174253)
+++ trunk/Source/WebInspectorUI/ChangeLog 2014-10-03 02:07:17 UTC (rev 174254)
@@ -1,3 +1,14 @@
+2014-10-02 Joseph Pecoraro <[email protected]>
+
+ Web Inspector: Cannot add a new CSS Rule to elements with 3 or more class names
+ https://bugs.webkit.org/show_bug.cgi?id=137375
+
+ Reviewed by Timothy Hatcher.
+
+ Correctly merge all class names with dots instead of just the first two.
+
+ * UserInterface/Models/DOMNode.js:
+
2014-10-02 Mark Lam <[email protected]>
Fixed the Inspector to be able to properly distinguish between scope types.
Modified: trunk/Source/WebInspectorUI/UserInterface/Models/DOMNode.js (174253 => 174254)
--- trunk/Source/WebInspectorUI/UserInterface/Models/DOMNode.js 2014-10-03 01:40:12 UTC (rev 174253)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/DOMNode.js 2014-10-03 02:07:17 UTC (rev 174254)
@@ -424,7 +424,7 @@
var className = this.getAttribute("class");
if (className) {
- var selector = "." + className.trim().replace(/\s+/, ".");
+ var selector = "." + className.trim().replace(/\s+/g, ".");
return (justSelector ? selector : lowerCaseName + selector);
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
