Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: static/css/main.css

Issue 29747647: Noissue - Document best practice for formatting multiline statements in Python (Closed)
Left Patch Set: Change names from camelCase to underscore_separated Created April 9, 2018, 11:23 p.m.
Right Patch Set: Incorporate Julian's suggestions Created April 12, 2018, 12:38 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Right: Side by side diff | Download
« no previous file with change/comment | « pages/coding-style.html ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
(no file at all)
1 * 1 *
2 { 2 {
3 font-family: Arial, sans; 3 font-family: Arial, sans;
4 font-size: 16px; 4 font-size: 16px;
5 } 5 }
6 6
7 body 7 body
8 { 8 {
9 margin: 0; 9 margin: 0;
10 /* Start below fixed navbar */ 10 /* Start below fixed navbar */
(...skipping 784 matching lines...) Expand 10 before | Expand all | Expand 10 after
795 left: 50%; 795 left: 50%;
796 width: 80px; 796 width: 80px;
797 margin-top: -60px; 797 margin-top: -60px;
798 margin-left: -40px; 798 margin-left: -40px;
799 } 799 }
800 800
801 .hide-disclaimer .video-play 801 .hide-disclaimer .video-play
802 { 802 {
803 margin-top: -40px; 803 margin-top: -40px;
804 } 804 }
805
806 .hide
807 {
808 display: none;
Sebastian Noack 2018/04/14 00:41:04 Can't we just use use the "hidden" HTML attribute:
juliandoucette 2018/04/14 13:17:42 The `hidden` attribute is not meant to be used in
Sebastian Noack 2018/04/14 13:24:30 Interesting. Then LGTM from my side. (Any reason y
809 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld