scss/_content.scss - Issue 29646555: Issue 6210 - Implement Subscription (Double-opt-in) template for newsletter

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

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

Side by Side Diff: scss/_content.scss

Issue 29646555: Issue 6210 - Implement Subscription (Double-opt-in) template for newsletter (Closed)

Patch Set: Update hgignore Created Dec. 21, 2017, 11:50 a.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

OLD	NEW
(Empty)
	1 /*******************************************************************************

	2 * Content

	3 ******************************************************************************/

	4

	5 a
	juliandoucette 2017/12/21 22:06:51 NIT: should be `.content a` if you are trying to b NIT: should be `.content a` if you are trying to be consistent with website-defaults? ire 2017/12/22 10:33:26 I'm not using the .content class here. This should Show quoted text On 2017/12/21 22:06:51, juliandoucette wrote: > NIT: should be `.content a` if you are trying to be consistent with > website-defaults? I'm not using the .content class here. This should apply to all links. juliandoucette 2017/12/22 13:32:23 You were at the time https://codereview.adblockplu Show quoted text On 2017/12/22 10:33:26, ire wrote: > I'm not using the .content class here. This should apply to all links. You were at the time https://codereview.adblockplus.org/29646555/diff/29646566/index.html#newcode11. Fine with me; anyway.
	6 {

	7 text-decoration: underline;

	8 }

	9

OLD	NEW