Make the strip line numbers link into a checkbox
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Dec 2012 01:35:36 +0000 (01:35 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Dec 2012 01:35:36 +0000 (01:35 +0000)
https://bugs.webkit.org/show_bug.cgi?id=104187

Reviewed by Tony Chang.

The link text was confusing.

* PrettyPatch/PrettyPatch.rb:
* code-review.js:

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

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
Websites/bugs.webkit.org/code-review.js

index 1be2544..8c15ead 100644 (file)
@@ -1,5 +1,17 @@
 2012-12-05  Ojan Vafai  <ojan@chromium.org>
 
+        Make the strip line numbers link into a checkbox
+        https://bugs.webkit.org/show_bug.cgi?id=104187
+
+        Reviewed by Tony Chang.
+
+        The link text was confusing.
+
+        * PrettyPatch/PrettyPatch.rb:
+        * code-review.js:
+
+2012-12-05  Ojan Vafai  <ojan@chromium.org>
+
         Sanitize content on copy in the code review tool
         https://bugs.webkit.org/show_bug.cgi?id=104155
 
index eeb0278..9561f5c 100644 (file)
@@ -417,6 +417,7 @@ h1 :hover {
   border: 0;
 }
 
+.LinkContainer label:after,
 .LinkContainer a:after {
   content: " | ";
   color: black;
@@ -426,6 +427,10 @@ h1 :hover {
   content: "";
 }
 
+.LinkContainer label {
+  color: #039;
+}
+
 .help {
  color: gray;
  font-style: italic;
@@ -501,7 +506,7 @@ div:focus {
 }
 </style>
 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js"></script> 
-<script src="code-review.js?version=44"></script>
+<script src="code-review.js?version=45"></script>
 </head>
 EOF
 
index 44c136a..7e0a13c 100644 (file)
@@ -1099,7 +1099,7 @@ var CODE_REVIEW_UNITTEST;
     $(document.body).prepend('<div id="message">' +
         '<div class="help">Select line numbers to add a comment. Scroll though diffs with the "j" and "k" keys.' +
           '<div class="DiffLinks LinkContainer">' +
-            '<a href="javascript:" id="line-number-on-copy-link"></a> ' +
+            '<input type="checkbox" id="line-number-on-copy"><label for="line-number-on-copy">Skip line numbers on copy</label>' +
             diffLinksHtml() +
           '</div>' +
           '<a href="javascript:" class="more">[more]</a>' +
@@ -1138,11 +1138,10 @@ var CODE_REVIEW_UNITTEST;
     Array.prototype.forEach.call(nodeList, callback);
   }
 
-  $('#line-number-on-copy-link').live('click', toggleShouldStripLineNumbersOnCopy);
+  $('#line-number-on-copy').live('click', toggleShouldStripLineNumbersOnCopy);
 
   function updateLineNumberOnCopyLinkContents() {
-    var link = document.getElementById('line-number-on-copy-link');
-    link.textContent = shouldStripLineNumbersOnCopy() ? 'Don\'t strip line numbers on copy' : 'Strip line numbers on copy';
+    document.getElementById('line-number-on-copy').checked = shouldStripLineNumbersOnCopy();
   }
 
   function shouldStripLineNumbersOnCopy() {