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
Issues
All Issues
Unified Diff: static/img/abb-logo.png
Issue
29841563
:
Noissue - Merged index_page into master (Closed)
Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Added Samsung Internet to browsers list for translations
Created July 30, 2018, 4:51 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
globals/browsers.py
includes/abb-cross-promotion.html
includes/abb-notification.tmpl
includes/as-mentioned.tmpl
includes/features.html
includes/footer.tmpl
includes/footnote.tmpl
includes/hero-download.html
includes/hero-download-conditional.tmpl
includes/index.tmpl
includes/navbar.tmpl
includes/styles.tmpl
locales/en/abb-notification.json
locales/en/footer.json
locales/en/menu.json
pages/android.md
pages/chrome.md
pages/download.tmpl
pages/edge.md
pages/firefox.md
pages/index.html
pages/internet-explorer.md
pages/maxthon.md
pages/opera.md
pages/safari.md
pages/yandex-browser.md
static/css/fonts.css
static/css/index.css
static/css/index-desktop.css
static/css/index-mobile.css
static/css/main.css
static/fonts/Source-Sans-Pro/300/cyrillic.woff2
static/fonts/Source-Sans-Pro/300/cyrillic-ext.woff2
static/fonts/Source-Sans-Pro/300/greek.woff2
static/fonts/Source-Sans-Pro/300/greek-ext.woff2
static/fonts/Source-Sans-Pro/300/latin.woff2
static/fonts/Source-Sans-Pro/300/latin-ext.woff2
static/fonts/Source-Sans-Pro/300/vietnamese.woff2
static/fonts/Source-Sans-Pro/400/cyrillic.woff2
static/fonts/Source-Sans-Pro/400/cyrillic-ext.woff2
static/fonts/Source-Sans-Pro/400/greek.woff2
static/fonts/Source-Sans-Pro/400/greek-ext.woff2
static/fonts/Source-Sans-Pro/400/latin.woff2
static/fonts/Source-Sans-Pro/400/latin-ext.woff2
static/fonts/Source-Sans-Pro/400/vietnamese.woff2
static/fonts/Source-Sans-Pro/700/cyrillic.woff2
static/fonts/Source-Sans-Pro/700/cyrillic-ext.woff2
static/fonts/Source-Sans-Pro/700/greek.woff2
static/fonts/Source-Sans-Pro/700/greek-ext.woff2
static/fonts/Source-Sans-Pro/700/latin.woff2
static/fonts/Source-Sans-Pro/700/latin-ext.woff2
static/fonts/Source-Sans-Pro/700/vietnamese.woff2
static/fonts/Source-Sans-Pro/LICENSE.txt
static/fonts/SourceSansPro-Light.woff
static/fonts/SourceSansPro-Regular.woff
static/fonts/SourceSansPro-Semibold.woff
static/img/abb-icon.png
static/img/abb-icon-2x.png
static/img/abb-logo.png
static/img/apple-app-store-badge.png
static/img/apple-app-store-badge.svg
static/img/device-desktop.png
static/img/device-desktop-2x.png
static/img/device-mobile.png
static/img/device-mobile.svg
static/img/download-icon.png
static/img/download-icon.svg
static/img/external-icon.png
static/img/external-icon.svg
static/img/feature-aa.png
static/img/feature-aa.svg
static/img/feature-fast.png
static/img/feature-fast.svg
static/img/feature-safe.png
static/img/feature-safe.svg
static/img/google-play-badge.png
static/img/google-play-badge.svg
static/img/hero-video.jpg
static/img/hero-video-2x.jpg
static/img/media/business-insider.png
static/img/media/business-insider-2x.png
static/img/media/mediapost.png
static/img/media/mediapost-2x.png
static/img/media/nyt.png
static/img/media/nyt-2x.png
static/img/media/techcrunch.png
static/img/media/techcrunch-2x.png
static/img/media/wsj.png
static/img/media/wsj-2x.png
static/img/video-thumbnail.jpg
static/js/index.js
static/js/vendor/bowser.js
View side-by-side diff with in-line comments
Download patch
« static/css/main.css
('K') |
« static/img/abb-icon-2x.png
('k') |
static/img/apple-app-store-badge.png »
('j') |
no next file with comments »
Expand Comments
('e') |
Collapse Comments
('c') |
Show Comments
Hide Comments
('s')
« static/css/main.css
('K') |
« static/img/abb-icon-2x.png
('k') |
static/img/apple-app-store-badge.png »
('j') |
no next file with comments »
Edit Message
Issue 29841563: Noissue - Merged index_page into master (Closed)
Created July 30, 2018, 3:47 a.m. by juliandoucette
Modified Aug. 3, 2018, 5:47 a.m.
Reviewers: ire
Base URL: https://hg.adblockplus.org/web.adblockplus.org
Comments: 23
This is Rietveld