Reviewed by Adam Roben.
authorweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 26 Jun 2007 04:18:26 +0000 (04:18 +0000)
committerweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 26 Jun 2007 04:18:26 +0000 (04:18 +0000)
        Patch for http://bugs.webkit.org/show_bug.cgi?id=14369
        HTML syntax highlighting colors and DOM tree colors should be the same

        * css/view-source.css:
        * page/inspector/ResourcePanel.js:
        * page/inspector/inspector.css:
        * page/inspector/inspector.js:
        * page/inspector/utilities.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@23780 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/css/view-source.css
WebCore/page/inspector/ResourcePanel.js
WebCore/page/inspector/inspector.css
WebCore/page/inspector/inspector.js
WebCore/page/inspector/utilities.js

index fd0ea44..6002a45 100644 (file)
@@ -1,3 +1,16 @@
+2007-06-25  Sam Weinig  <sam@webkit.org>
+
+        Reviewed by Adam Roben.
+
+        Patch for http://bugs.webkit.org/show_bug.cgi?id=14369
+        HTML syntax highlighting colors and DOM tree colors should be the same
+
+        * css/view-source.css:
+        * page/inspector/ResourcePanel.js:
+        * page/inspector/inspector.css:
+        * page/inspector/inspector.js:
+        * page/inspector/utilities.js:
+
 2007-06-25  Adam Roben  <aroben@apple.com>
 
         Fix Bug 14402: Main resource of http://www.apple.com/safari/ is called "www.apple.com" instead of "safari"
index 686bf12..866054b 100644 (file)
@@ -76,31 +76,37 @@ td {
 }
 
 .webkit-html-tag {
-    color: rgb(136, 18, 128)
+    /* Keep this in sync with inspector.css (.webkit-html-tag) */
+    color: rgb(136, 18, 128);
 }
 
 .webkit-html-attribute-name {
-    color: rgb(153, 69, 0)
+    /* Keep this in sync with inspector.css (.webkit-html-attribute-name) */
+    color: rgb(153, 69, 0);
 }
 
 .webkit-html-attribute-value {
-    color: rgb(26, 26, 166)
+    /* Keep this in sync with inspector.css (.webkit-html-attribute-value) */
+    color: rgb(26, 26, 166);
+}
+
+.webkit-html-external-link, .webkit-html-resource-link {
+    /* Keep this in sync with inspector.css (.webkit-html-external-link, .webkit-html-resource-link) */
+    color: #00e;
 }
 
 .webkit-html-external-link {
+    /* Keep this in sync with inspector.css (.webkit-html-external-link) */
     text-decoration: none;
 }
 
 .webkit-html-external-link:hover {
+    /* Keep this in sync with inspector.css (.webkit-html-external-link:hover) */
     text-decoration: underline;
 }
 
-/*
-.webkit-html-resource-link {
-}
-*/
-
 .webkit-html-comment {
+    /* Keep this in sync with inspector.css (.comment) */
     color: rgb(35, 110, 37);
 }
 
index fa9a8f3..871dd83 100644 (file)
@@ -1190,7 +1190,7 @@ WebInspector.DOMNodeTreeElement.populate = function(element)
     }
 
     if (element.representedObject.nodeType == Node.ELEMENT_NODE) {
-        var title = "<span class=\"tag close\">&lt;/" + element.representedObject.nodeName.toLowerCase().escapeHTML() + "&gt;</span>";
+        var title = "<span class=\"webkit-html-tag close\">&lt;/" + element.representedObject.nodeName.toLowerCase().escapeHTML() + "&gt;</span>";
         var item = new TreeElement(title, element.representedObject, false);
         item.selectable = false;
         element.appendChild(item);
index 13702f0..f826ba0 100644 (file)
@@ -880,7 +880,7 @@ body.inactive #sidebar li.selected {
     text-indent: -12px
 }
 
-.content.tree li .tag.close {
+.content.tree li .webkit-html-tag.close {
     margin-left: -12px;
 }
 
@@ -905,31 +905,38 @@ body.inactive #sidebar li.selected {
     display: block;
 }
 
-.content.tree li .comment, body.inactive .content.tree li .comment {
-    color: rgb(0, 116, 0);
+.webkit-html-comment {
+    /* Keep this in sync with view-source.css (.webkit-html-comment) */
+    color: rgb(35, 110, 37);
 }
 
-.content.tree li .tag, body.inactive .content.tree li .tag {
-    color: rgb(170, 13, 145);
+.webkit-html-tag {
+    /* Keep this in sync with view-source.css (.webkit-html-tag) */
+    color: rgb(136, 18, 128);
 }
 
-.content.tree li .attr, body.inactive .content.tree li .attr {
-    color: rgb(131, 108, 40);
+.webkit-html-attribute-name {
+    /* Keep this in sync with view-source.css (.webkit-html-attribute-name) */
+    color: rgb(153, 69, 0);
 }
 
-.content.tree li .value, body.inactive .content.tree li .value {
-    color: rgb(196, 26, 22);
+.webkit-html-attribute-value {
+    /* Keep this in sync with view-source.css (.webkit-html-attribute-value) */
+    color: rgb(26, 26, 166);
 }
 
-.webkit-html-external-link, -webkit-html-resource-link {
+.webkit-html-external-link, .webkit-html-resource-link {
+    /* Keep this in sync with view-source.css (.webkit-html-external-link, .webkit-html-resource-link) */
     color: #00e;
 }
 
 .webkit-html-external-link {
+    /* Keep this in sync with view-source.css (.webkit-html-external-link) */
     text-decoration: none;
 }
 
 .webkit-html-external-link:hover {
+    /* Keep this in sync with view-source.css (.webkit-html-external-link:hover) */
     text-decoration: underline;
 }
 
index 41dafe3..6b1e0a2 100644 (file)
@@ -696,12 +696,13 @@ WebInspector.showResourceForURL = function(url)
     return true;
 }
 
-WebInspector.linkifyURL = function(url, linkText, isExternal)
+WebInspector.linkifyURL = function(url, linkText, classes, isExternal)
 {
     if (linkText === undefined)
         linkText = url.escapeHTML();
-    var className = isExternal ? "webkit-html-external-link" : "webkit-html-resource-link";
-    var link = "<a href=\"" + url + "\" class=\"" + className + "\" target=\"_blank\">" + linkText + "</a>";
+    classes = (classes === undefined) ? "" : classes + " ";
+    classes += isExternal ? "webkit-html-external-link" : "webkit-html-resource-link";
+    var link = "<a href=\"" + url + "\" class=\"" + classes + "\" target=\"_blank\">" + linkText + "</a>";
     return link;
 }
 
index e725919..2aab4f3 100644 (file)
@@ -509,18 +509,20 @@ function nodeTitleInfo(hasChildren, linkify)
             break;
 
         case Node.ELEMENT_NODE:
-            info.title = "<span class=\"tag\">&lt;" + this.nodeName.toLowerCase().escapeHTML();
+            info.title = "<span class=\"webkit-html-tag\">&lt;" + this.nodeName.toLowerCase().escapeHTML();
 
             if (this.hasAttributes()) {
                 for (var i = 0; i < this.attributes.length; ++i) {
                     var attr = this.attributes[i];
                     var value = attr.value.escapeHTML();
-                    value = value.replace(/([\/;:\)\]\}])/g, "$1&#8203;")
-                    if (linkify && (attr.name === "src" || attr.name === "href"))
-                        value = linkify(attr.value, value, this.nodeName.toLowerCase() == "a");
+                    value = value.replace(/([\/;:\)\]\}])/g, "$1&#8203;");
+
+                    info.title += " <span class=\"webkit-html-attribute-name\">" + attr.name.escapeHTML() + "</span>=&#8203;";
 
-                    info.title += " <span class=\"attr\">" + attr.name.escapeHTML() + "</span>=&#8203;";
-                    info.title += "<span class=\"value\">\"" + value + "\"</span>";
+                    if (linkify && (attr.name === "src" || attr.name === "href"))
+                        info.title += linkify(attr.value, value, "webkit-html-attribute-value", this.nodeName.toLowerCase() == "a");
+                    else
+                        info.title += "<span class=\"webkit-html-attribute-value\">\"" + value + "\"</span>";
                 }
             }
             info.title += "&gt;</span>&#8203;";
@@ -533,7 +535,7 @@ function nodeTitleInfo(hasChildren, linkify)
             var showInlineText = textChild && textChild.textContent.length < Preferences.maxInlineTextChildLength;
 
             if (showInlineText) {
-                info.title += textChild.nodeValue.escapeHTML() + "&#8203;<span class=\"tag\">&lt;/" + this.nodeName.toLowerCase().escapeHTML() + "&gt;</span>";
+                info.title += textChild.nodeValue.escapeHTML() + "&#8203;<span class=\"webkit-html-tag\">&lt;/" + this.nodeName.toLowerCase().escapeHTML() + "&gt;</span>";
                 info.hasChildren = false;
             }
             break;
@@ -546,7 +548,7 @@ function nodeTitleInfo(hasChildren, linkify)
             break
 
         case Node.COMMENT_NODE:
-            info.title = "<span class=\"comment\">&lt;!--" + this.nodeValue.escapeHTML() + "--&gt;</span>";
+            info.title = "<span class=\"webkit-html-comment\">&lt;!--" + this.nodeValue.escapeHTML() + "--&gt;</span>";
             break;
 
         default: