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

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

Issue 29841563: Noissue - Merged index_page into master (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Left Patch Set: Added Samsung Internet to browsers list for translations Created July 30, 2018, 4:51 a.m.
Right Patch Set: Fixed more misc issues discussed in IRC Created July 30, 2018, 4:40 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
LEFTRIGHT
1 body 1 body
2 { 2 {
3 font-family: "Source Sans Pro", Arial, sans-serif; 3 font-family: "Source Sans Pro", Arial, sans-serif;
4 margin: 0; 4 margin: 0;
5 /* Start below fixed navbar */ 5 /* Start below fixed navbar */
6 padding-top: 4em; 6 padding-top: 4em;
7 line-height: 1.5; 7 line-height: 1.5;
8 } 8 }
9 9
10 #content, 10 #content,
(...skipping 240 matching lines...) Expand 10 before | Expand all | Expand 10 after
251 251
252 /******************************************************************************* 252 /*******************************************************************************
253 * .fg-* utilities 253 * .fg-* utilities
254 ******************************************************************************/ 254 ******************************************************************************/
255 255
256 .fg-accent 256 .fg-accent
257 { 257 {
258 color: #077CA6; 258 color: #077CA6;
259 } 259 }
260 260
261 .bg-secondary 261 .bg-secondary
ire 2018/07/31 00:10:10 NIT: This is a bg-* utility, not fg-*
juliandoucette 2018/08/02 20:51:56 Good catch :D
262 { 262 {
263 color: #000; 263 color: #000;
264 background-color: #F4F4F4; 264 background-color: #F4F4F4;
265 } 265 }
266 266
267 /******************************************************************************* 267 /*******************************************************************************
268 * #navbar 268 * #navbar
269 ******************************************************************************/ 269 ******************************************************************************/
270 270
271 #navbar 271 #navbar
(...skipping 531 matching lines...) Expand 10 before | Expand all | Expand 10 after
803 { 803 {
804 margin-top: -60px; 804 margin-top: -60px;
805 } 805 }
806 } 806 }
807 807
808 .hide-disclaimer .video-play 808 .hide-disclaimer .video-play
809 { 809 {
810 margin-top: -40px; 810 margin-top: -40px;
811 } 811 }
812 812
813 .hide 813 .hide
juliandoucette 2018/07/30 05:01:06 Detail: I decided to keep this class while merging
ire 2018/07/31 00:10:04 I checked and it is being used on at least the cod
juliandoucette 2018/08/02 20:51:54 Thanks!
814 { 814 {
815 display: none; 815 display: none;
816 } 816 }
817 817
818 /****************************************************************************** 818 /******************************************************************************
819 * .item-group 819 * .item-group
820 *****************************************************************************/ 820 *****************************************************************************/
821 821
822 .item-group 822 .item-group
823 { 823 {
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
913 .content .horizontal-list 913 .content .horizontal-list
914 { 914 {
915 padding: 0; 915 padding: 0;
916 } 916 }
917 917
918 .horizontal-list li 918 .horizontal-list li
919 { 919 {
920 display: inline; 920 display: inline;
921 list-style: none; 921 list-style: none;
922 } 922 }
923
924 /* Download button
925 ******************************************************************************/
926
927 .content .tos-short
928 {
929 margin-bottom: 0.5em;
930 }
931
932 .content .install-button
933 {
934 display: inline-block;
935 min-width: 250px;
936 /* stretch full-width <= small tablets */
937 width: 100%;
938 /* 0.2em taller than normal buttons */
939 padding-top: 0.8em;
940 padding-bottom: 0.8em;
941 color: #fff;
ire 2018/07/31 00:10:12 We also need to style the :visited pseudo-class be
juliandoucette 2018/08/02 20:51:57 Good catch!
942 background-color: #3D9C4F;
943 }
944
945 .content .install-button:hover,
946 .content .install-button:focus
947 {
948 background-color: #258333;
949 }
950
951 @media (min-width: 576px)
952 {
953 .content .install-button
954 {
955 /* force auto-width >= small tablets */
956 width: auto;
957 }
958 }
959
960 .content .download-alt
961 {
962 font-size: smaller;
963 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld