Keyboard Shortcuts

File
u :up to issue
m :publish + mail comments
M :edit review message
j / k :jump to file after / before current file
J / K :jump to next file with a comment after / before current file
Side-by-side diff
i :toggle intra-line diffs
e :expand all comments
c :collapse all comments
s :toggle showing all comments
n / p :next / previous diff chunk or comment
N / P :next / previous comment
<Up> / <Down> :next / previous line
<Enter> :respond to / edit current comment
d :mark current comment as done
Issue
u :up to list of issues
m :publish + mail comments
j / k :jump to patch after / before current patch
o / <Enter> :open current patch in side-by-side view
i :open current patch in unified diff view
Issue List
j / k :jump to issue after / before current issue
o / <Enter> :open current issue
# : close issue
Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :save comment
<Esc> :cancel edit
Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(382)
Issues Repositories Search
Open Issues | Closed Issues | All Issues | Sign in with your Google Account to create issues and add comments

Delta Between Two Patch Sets: static/script.js

Issue 3098041: Hook up implementation of showing a mildly prettified unified diff on the issue
Left Patch Set: Created 15 years, 2 months ago
Right Patch Set: Fix link logic to use onclick instead of href Created 15 years, 1 month ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « codereview/views.py ('k') | templates/patchset.html » ('j') | no next file with change/comment »
('i') | ('e') | ('c') | ('s')
LEFTRIGHT
1 // Copyright 2008 Google Inc. 1 // Copyright 2008 Google Inc.
2 // 2 //
3 // Licensed under the Apache License, Version 2.0 (the "License"); 3 // Licensed under the Apache License, Version 2.0 (the "License");
4 // you may not use this file except in compliance with the License. 4 // you may not use this file except in compliance with the License.
5 // You may obtain a copy of the License at 5 // You may obtain a copy of the License at
6 // 6 //
7 // http://www.apache.org/licenses/LICENSE-2.0 7 // http://www.apache.org/licenses/LICENSE-2.0
8 // 8 //
9 // Unless required by applicable law or agreed to in writing, software 9 // Unless required by applicable law or agreed to in writing, software
10 // distributed under the License is distributed on an "AS IS" BASIS, 10 // distributed under the License is distributed on an "AS IS" BASIS,
(...skipping 2191 matching lines...) | | Loading...
2202 var child = dashboardState.curTR.cells[3].firstChild; 2202 var child = dashboardState.curTR.cells[3].firstChild;
2203 while (child && child.nextSibling && child.nodeName != "A") { 2203 while (child && child.nextSibling && child.nodeName != "A") {
2204 child = child.nextSibling; 2204 child = child.nextSibling;
2205 } 2205 }
2206 if (child && child.nodeName == "A") { 2206 if (child && child.nodeName == "A") {
2207 location.href = child.href; 2207 location.href = child.href;
2208 } 2208 }
2209 } 2209 }
2210 } else if (key == 'I') { 2210 } else if (key == 'I') {
2211 if (dashboardState) { 2211 if (dashboardState) {
2212 » var child = dashboardState.curTR.cells[2].firstChild; 2212 var child = dashboardState.curTR.cells[2].firstChild;
2213 » while (child && child.nextSibling && 2213 while (child && child.nextSibling &&
2214 » (child.nodeName != "A" || child.style.display == "none" || child. href.indexOf("javascript") != 0)) { 2214 (child.nodeName != "A" || child.style.display == "none" ||
2215 » child = child.nextSibling; 2215 !child.onclick)) {
2216 » } 2216 child = child.nextSibling;
2217 » if (child && child.nodeName == "A") { 2217 }
2218 » location.href = child.href; 2218 if (child && child.nodeName == "A") {
2219 » } 2219 child.onclick();
2220 }
2220 } 2221 }
2221 } else if (key == 'K') { 2222 } else if (key == 'K') {
2222 if (dashboardState) dashboardState.gotoPrev(); 2223 if (dashboardState) dashboardState.gotoPrev();
2223 } else if (key == 'J') { 2224 } else if (key == 'J') {
2224 if (dashboardState) dashboardState.gotoNext(); 2225 if (dashboardState) dashboardState.gotoNext();
2225 } else if (key == 'M') { 2226 } else if (key == 'M') {
2226 document.location.href = publish_link; 2227 document.location.href = publish_link;
2227 } else if (key == 'U') { 2228 } else if (key == 'U') {
2228 // back to dashboard 2229 // back to dashboard
2229 document.location.href = base_url; 2230 document.location.href = base_url;
(...skipping 916 matching lines...) | | Loading...
3146 httpreq.send(""); 3147 httpreq.send("");
3147 } 3148 }
3148 3149
3149 /** 3150 /**
3150 * Helper function that returns the dialog's HTML container. 3151 * Helper function that returns the dialog's HTML container.
3151 */ 3152 */
3152 M_draftMessage.prototype.get_dialog_ = function() { 3153 M_draftMessage.prototype.get_dialog_ = function() {
3153 return document.getElementById(this.id_dlg_container); 3154 return document.getElementById(this.id_dlg_container);
3154 } 3155 }
3155 3156
LEFTRIGHT
« codereview/views.py ('k') | templates/patchset.html » ('j') | ('i') | ('e') | ('c') | ('s')
Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b

AltStyle によって変換されたページ (->オリジナル) /