Any chance we can get this CSS minified? It's one of the things that keeps coming up as a knock on PageSpeed Insights.

From Ron: ENHC0014649

I'd like to explore effort it would take to add another option to BU Slideshow plugin to put the photo headline and caption UNDER the image, rather than overlaying those elements on top of the image.

The CSS selectors here are long and difficult for people coding themes to override. Simplifying them shouldn't affect sites or custom sites as long as the HTML doesn't change, since more specific selectors will always override less specific ones. Our coding standards say that no more than three levels of nesting are allowed, and nesting should be avoided at all costs.

Here's an example: https://github.com/bu-ist/bu-slideshow/blob/master/interface/css/bu-slideshow.css#L422

Let's get these cleaned up! 👍

Found while trying to save. /wp-admin/admin.php?page=bu-add-slideshow&noheader

Notice: Undefined index: bu_slideshow_template in /Users/awbauer/Sites/wp/wp-content/plugins/bu-slideshow/bu-slideshow.php on line 547

In WP 4.1.9, modal height is too short.

TY @gjensen01 for the report.

For text such as Test text   123 ©, don't render out the entities. Show as originally submitted.

Also tracking in SN as ENHC0013874.

Right now, bu_slideshow_slide_html only runs on the public slideshow view. It'd be great if it ran on the admin view as well, so we have control over the admin UI for custom templates too. Quick test below:

function test_admin_template( $html, $slide ){
    if( "admin" == $slide->view ) {
        $output =  "<div>Doing a thing in the admin view!</div>" . $html;
        return $output;
    }
    if( "public" == $slide->view ) {
        $output =  "<div>Doing a thing in the public view!</div>" . $html;
        return $output;
    }
    return $html;
}

add_filter( 'bu_slideshow_slide_html', 'test_admin_template', 10, 2 );

Changing slide order results in caption positions on some slides getting unset. Scope limited to individual slides. Issue believed to stem from https://core.trac.wordpress.org/ticket/16972

Ex:

1. Reorder slides A, B, C, D, E, F to A, B, D, C, E, F.

• In this case: A, B, C, D & F wouldn't affected, but E would be.

2. Reorder slides A, B, C, D, E, F to B, A, C, E, D, F

• In this case: B, C, D & E wouldn't be affected, but A & F would be.

Workaround: Reset the position on the affected slides. Confirmed that no other data is affected by the issue.

Ref: INC11613193

/bu-slideshow/interface/js/bu-slideshow-admin.js: jQuery.fn.bind() is deprecated

It would be great if you could duplicate a slideshow and edit the duplicate so that clients can keep their original setup, but have a second slideshow available to make changes or swap out when needed.

Found on /wp-admin/admin.php?page=bu-add-slideshow

Notice: Undefined variable: valid_templates in /Users/awbauer/Sites/wp/wp-content/plugins/bu-slideshow/interface/edit-slideshow-form.php on line 6

Inserting a slideshow through the modal doesn't add the shuffle=true option into the shortcode.

ty @samhuntley for reporting.

PHP Notice:

Notice: Use of undefined constant BU_Slideshow - assumed 'BU_Slideshow' in .../wp-content/plugins/bu-slideshow/bu-slideshow.php on line 1111

https://github.com/bu-ist/bu-slideshow/blob/master/bu-slideshow.php#L1111

Custom fields should use wp_kses() instead of sanitize_text_field() to allow HTML to be used.

ty @gjensen01 for reporting.

With the Gutenberg block editor the 'wp-content-editor-tools' area is gone. This is the area where the "Add Media" button as well as the 'Add Slideshow" and "Add Form" (for the Gravity Forms plugin) buttons were located on the post/page edit screen.

Slideshows can still be included using the 'Shortcode' Block. Ultimately we need to create a Custom Slideshow Block so users can add slideshows with gutenberg's Add Block UI.

Hey there! I was setting up a slideshow for a client today, and there are a couple of things that I think would make this plugin much easier for me (and our clients) to edit:

• Many slideshows that I've been building don't need a slide title for each slide. It would be very helpful if the Title field was left blank by default instead of filled with Untitled Slide. For every slideshow I've built so far, I've had to delete that placeholder for every single slide.
• An option to create slides from a bulk upload of photos would be very helpful and save a ton of time, so that I don't need to create each slide individually. A lot of clients ask for a page full of photos and don't send any titles or captions, and giving them an option to automatically make a slideshow from a bunch of photos in the Wordpress admin would help them a lot (and keep them from making page after page of nothing but photos)
• Moving the "Add new slide" button underneath the last slide would make it easier to edit the new slides that are added, since you will always be close to the slide that's being added. Right now, after two or three slides are added, you have to scroll back and forth every time you add a new slide to edit that slide and add another.

There's probably other improvements that could be made to the UI, but I think these are the things that would make the biggest difference in making this feel faster and easier to use.

Thanks!

(via @ronyeany)

On the Edit Slideshow screen, expand/collapse arrows don't change when div is opened or collapsed.