INN / umbrella-borderzine

Umbrella repository for borderzine.com/
GNU General Public License v2.0
0 stars 1 forks source link

Add 6 column grid widget for photo galleries section #75

Closed joshdarby closed 4 years ago

joshdarby commented 4 years ago

Changes

This pull request makes the following changes:

Screen Shot 2019-10-07 at 4 00 31 PM

Why

For #36

Testing/Questions

Features that this PR affects:

Questions that need to be answered before merging:

Steps to test this PR:

  1. Add the Borderzine 6 Column Widget widget to the Series Life on the Borderline: Footer widget area
    • Title: Photo Galleries
    • Avoid Duplicate Posts
    • Large image
    • Excerpt display: none
    • Hide the publish date
    • Limit to category: Multimedia (?)
  2. View widget on frontend and verify that it looks ok

Additional information

INN Member/Labs Client requesting: (if applicable)

benlk commented 4 years ago

The code changes in f766833 make sense, but I'm presently experiencing difficulties testing them.

I'm not seeing the widget output on my local site, and even with WP_DEBUG to true there's no output upon the page or in the log. It looks like the widget isn't saving?

joshdarby commented 4 years ago

As a note, https://github.com/INN/umbrella-borderzine/pull/75/commits/8aad4742d0fe6b2ce24a77d6644d365eeba9a2fc sets the widget $context to set podcast => false so no play icons appear due to https://github.com/INN/umbrella-citylimits/issues/87