From 7f67b8e5c25654114ced7408a0dc58573f070bf2 Mon Sep 17 00:00:00 2001
From: Jeremy Herve <jeremy@tagada.hu>
Date: Thu, 20 Dec 2018 08:31:11 +0100
Subject: [PATCH] Twenty Nineteen: updates to widget styles (#10978)

Related: #10335

#### Changes proposed in this Pull Request:
<!--- Explain what functional changes your PR includes -->

* Bring in additional widget CSS adjustments, already committed on WordPress.com (see D22275-code)
* Add Twenty Nineteen stylesheet to the AutoRTL build, so a `twentynineteen-rtl.css` file is generated on build and for release.

#### Testing instructions:

* Checkout this branch on a site using Twenty Nineteen.
* Make sure all widgets (especially those with lists) look good.
* Run `gulp frontendcss:separate`
* Make sure you see a new `twentynineteen-rtl.css` in `modules/theme-tools/compat/`

#### Proposed changelog entry for your changes:

* Twenty Nineteen: additional style adjustments to make sure all Jetpack widgets look good with the theme.
---
 modules/theme-tools/compat/twentynineteen.css | 67 ++++++++++++++++---
 tools/builder/frontend-css.js                 |  1 +
 2 files changed, 57 insertions(+), 11 deletions(-)

diff --git a/modules/theme-tools/compat/twentynineteen.css b/modules/theme-tools/compat/twentynineteen.css
index 2bd0512c7f2b7..53519afa7607a 100644
--- a/modules/theme-tools/compat/twentynineteen.css
+++ b/modules/theme-tools/compat/twentynineteen.css
@@ -228,6 +228,30 @@
  * Widgets
  */
 
+/* Widget List Resets */
+.widget_authors ul,
+.widget_author_grid ul,
+.widget_jp_blogs_i_follow ul,
+.widget_links ul,
+.widget_rss_links ul {
+	list-style: none;
+	padding-left: 0;
+	padding-right: 0;
+}
+
+.widget_links li,
+.widget_jp_blogs_i_follow li,
+.widget_rss_links li {
+	color: #767676;
+	font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", "Roboto", "Oxygen", "Ubuntu", "Cantarell", "Fira Sans", "Droid Sans", "Helvetica Neue", sans-serif;
+	font-size: calc(22px * 1.125);
+	font-weight: 700;
+	line-height: 1.2;
+	margin-top: 0.5rem;
+	margin-bottom: 0.5rem;
+}
+
+
 /* Authors Widget */
 .widget.widget_authors ul li > ul {
 	list-style-type: disc;
@@ -297,19 +321,40 @@
 	padding: 10px 12px;
 }
 
-/* RSS Links */
-.widget_rss_links ul {
-	list-style: none;
-	padding-left: 0;
-	padding-right: 0;
+/* RSS Feed Widget */
+.widget_rss li a.rsswidget {
+	display: block;
+	margin: 1rem 0 0.5rem;
 }
 
-.widget_rss_links li {
-	font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", "Roboto", "Oxygen", "Ubuntu", "Cantarell", "Fira Sans", "Droid Sans", "Helvetica Neue", sans-serif;
-	font-size: calc(22px * 1.125);
-	font-weight: bold;
-	line-height: 1.2;
-	padding-bottom: 0.75rem;
+.widget_rss .rssSummary {
+	font-family: "NonBreakingSpaceOverride", "Hoefler Text", "Baskerville Old Face", Garamond, "Times New Roman", serif;
+	font-weight: 400;
+	font-size: 22px;
+}
+
+.widget_rss cite,
+.widget_rss .rss-date {
+	color: #9c9c9c;
+	display: block;
+	font-size: 0.71111em;
+	font-weight: 500;
+	margin: 0.5rem 0;
+}
+
+.widget_rss cite {
+	font-style: normal;
+}
+
+.widget_rss cite:before {
+	content: "\2014\00a0";
+}
+
+/* Search widget override */
+@media only screen and (min-width: 600px) {
+	.widget.widget_search .search-field {
+		max-width: calc( 50vw - 20% );
+	}
 }
 
 /**
diff --git a/tools/builder/frontend-css.js b/tools/builder/frontend-css.js
index 42fad6dcbd60a..8be0e8e451e4f 100644
--- a/tools/builder/frontend-css.js
+++ b/tools/builder/frontend-css.js
@@ -63,6 +63,7 @@ const separate_list = [
 	'modules/shortcodes/css/recipes.css',
 	'modules/shortcodes/css/recipes-print.css',
 	'modules/tiled-gallery/tiled-gallery/tiled-gallery.css',
+	'modules/theme-tools/compat/twentynineteen.css',
 ];
 
 const pathModifier = function( file, contents ) {