README.md - Issue 29510560: Issue 4633 - Default form styles

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: README.md

Issue 29510560: Issue 4633 - Default form styles (Closed) Base URL: https://hg.adblockplus.org/website-defaults

Patch Set: Addressed NITs Created Aug. 11, 2017, 3:23 p.m.

Left:
Right:

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

Jump to:

OLD	NEW
1 # Website defaults	1 # Website defaults

2	2

3 Default resources for eyeo websites.	3 Default resources for eyeo websites.

	4

	5 ## License

	6

	7 - normalize.css: [MIT License](https://github.com/necolas/normalize.css/blob/mas ter/LICENSE.md) by Nicolas Gallagher and Jonathan Neal
	juliandoucette 2017/08/24 12:35:23 Shouldn't we mention that our CSS contains part of Shouldn't we mention that our CSS contains part of normalize.css (or equivalent)? This reads like we actually included normalize.css as part of our code. ire 2017/09/13 07:59:56 Done. Show quoted text On 2017/08/24 12:35:23, juliandoucette wrote: > Shouldn't we mention that our CSS contains part of normalize.css (or > equivalent)? This reads like we actually included normalize.css as part of our > code. Done.
OLD	NEW

« no previous file with comments | « no previous file | pages/forms/advanced.html » ('j') | scss/forms/_advanced.scss » ('J')