Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theme Tools / Infinite Scroll: add support for Twenty Nineteen #10335

Closed
8 of 10 tasks
jeherve opened this issue Oct 16, 2018 · 5 comments
Closed
8 of 10 tasks

Theme Tools / Infinite Scroll: add support for Twenty Nineteen #10335

jeherve opened this issue Oct 16, 2018 · 5 comments
Assignees
Labels
Epic Formerly "Primary Issue", or "Master Issue" [Feature] Infinite Scroll [Feature] Theme Tools [Pri] High [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Milestone

Comments

@jeherve
Copy link
Member

jeherve commented Oct 16, 2018

Twenty Nineteen is expected to ship with WordPress 5.0 in November. It is currently being developed here:
https://github.com/WordPress/twentynineteen

In the past, and with previous default themes, we've added a number of compatibility files and functions to make sure new WordPress site owners had the best experience they could get when using the default theme and Jetpack. In practice, it meant adding the following:

There are also things we haven't done for previous default themes, but may want to do for this one:

  • Add Social Menu Support
  • Add Content Options support

See how it was done for Twenty Sixteen for example: #2970

@jeherve jeherve added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] Infinite Scroll [Feature] Theme Tools [Pri] High labels Oct 16, 2018
@jeherve jeherve added this to the 6.7 milestone Oct 16, 2018
@davidakennedy
Copy link

Hey @jeherve, thanks for creating this. @allancole will be working on it as part of his work on Twenty Nineteen.

One question I have is around the "Alter Gallery widget default width." Why would that be necessary? Have we needed to do that before?

@jeherve
Copy link
Member Author

jeherve commented Oct 18, 2018

This is probably something that we could drop for this theme; it is less important now.

As a bit of background, Jetpack used to have its own Gallery widget, and that widget had a default width setting that could be overwritten via a filter to match the exact width of your sidebar. Here is how it looked for Twenty Sixteen for example:

/**
* Alter gallery widget default width.
*/
function twentysixteen_gallery_widget_content_width( $width ) {
return 390;
}
add_filter( 'gallery_widget_content_width', 'twentysixteen_gallery_widget_content_width' );

WordPress Core then introduced its own Gallery widget in WP 4.9, and we worked to migrate all old Jetpack Gallery widgets to the new Core Gallery widget (related PR: #8062). Only folks running WP 4.8 or forcing the old Jetpack widget on via this filter now use the old Jetpack Gallery Widget. It consequently may not be worth adding compatibility to that old widget in the new default theme. It's only a few lines where you define the max width of your sidebar though, but we may not need that at all. I'll let you make that call. 👍

allancole added a commit that referenced this issue Oct 24, 2018
Adds support for infinite-scroll, jetpack-responsive-videos, jetpack-geo-location, jetpack-content-options, and jetpack-widgets, more to come...

See: #10335
jeherve pushed a commit that referenced this issue Oct 31, 2018
<!--- Provide a general summary of your changes in the Title above -->

This PR addresses issue #10335. It still needs a little more work as outlined below.

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

- [x] Add Infinite Scroll support and tweak CSS to make sure the IS buttons look good.
- [x] Add support for Responsive Videos.
- [x] Add Content Options support for author-bio, blog-display, post-details, and featured-images 
- [x] Alter Gallery widget default width (possibly not necessary unless using older versions of WP).
- [ ] Style widgets and shortcodes to match the style of the theme (coming soon).
- [ ] Add Social Menu Support (requires a pending code refactor in theme, may not be possible).

#### Testing instructions:
<!-- Please include detailed testing steps, explaining how to test your change. -->
<!-- Bear in mind that context you working on is not obvious for everyone.  -->
<!-- Adding "simple" configuration steps will help reviewers to get to your PR as quickly as possible. -->
<!-- "Before / After" screenshots can also be very helpful when the change is visual. -->

* Test the theme against all the customizer and Jetpack settings mentioned above 
* **Note**: requires latest `master` version of Twenty Nineteen from Github here: https://github.com/WordPress/twentynineteen/

#### Proposed changelog entry for your changes:
<!-- Please do not leave this empty. If no changelog entry needed, state as such. -->

* Add compatibility for Twenty Nineteen default theme
@jeherve
Copy link
Member Author

jeherve commented Nov 2, 2018

Here are some related issues: #10513 #10522

@jeherve jeherve added the Epic Formerly "Primary Issue", or "Master Issue" label Nov 2, 2018
@jeherve jeherve modified the milestones: 6.7, 6.8 Nov 2, 2018
@crunnells
Copy link
Contributor

crunnells commented Nov 7, 2018

@jeherve Both of those issues (#10513 and #10522) should be resolved now, and we'll submit a PR after Allan does a final review.

@jeherve
Copy link
Member Author

jeherve commented Nov 15, 2018

One item remains on that list, but we can tackle it later on:
#10485

jeherve added a commit that referenced this issue Dec 20, 2018
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic Formerly "Primary Issue", or "Master Issue" [Feature] Infinite Scroll [Feature] Theme Tools [Pri] High [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Development

No branches or pull requests

4 participants