Skip to content

Commit

Permalink
Twenty Nineteen: updates to widget styles (#10978)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
jeherve authored Dec 20, 2018
1 parent cbd728c commit 7f67b8e
Show file tree
Hide file tree
Showing 2 changed files with 57 additions and 11 deletions.
67 changes: 56 additions & 11 deletions modules/theme-tools/compat/twentynineteen.css
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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% );
}
}

/**
Expand Down
1 change: 1 addition & 0 deletions tools/builder/frontend-css.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 ) {
Expand Down

0 comments on commit 7f67b8e

Please sign in to comment.