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

Add Twenty Sixteen Compat Files #2970

Merged
merged 3 commits into from
Nov 13, 2015
Merged

Add Twenty Sixteen Compat Files #2970

merged 3 commits into from
Nov 13, 2015

Conversation

iamtakashi
Copy link
Contributor

Theme compat files for Twenty Sixteen. These include...

  • Add support for Responsive Videos.
  • Alter Gallery widget default width.
  • Remove Sharing and Likes from custom excerpt because the excerpt is used as intro in the theme.
  • Add support and style tweak for Infinite Scroll.
  • Style front-end modules to match the style of the theme.

@kraftbj kraftbj added this to the 3.8.1 milestone Nov 6, 2015
@kraftbj kraftbj added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it General [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. labels Nov 6, 2015
@jeherve jeherve added the tdiv label Nov 6, 2015
*/
function twentysixteen_remove_share() {
if ( has_excerpt() ) {
remove_filter( 'the_excerpt', 'sharing_display',19 );
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nitpick: Missing a space before 19

@dereksmart
Copy link
Member

Gave this a spin, everything is coming up roses.

2016 is beautiful, @iamtakashi!

@dereksmart dereksmart added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. labels Nov 13, 2015
@michaeldcain
Copy link
Member

LGTM :shipit:

dereksmart added a commit that referenced this pull request Nov 13, 2015
@dereksmart dereksmart merged commit c0bc689 into Automattic:master Nov 13, 2015
@dereksmart dereksmart removed the [Status] Ready to Merge Go ahead, you can push that green button! label Nov 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants