sesecrets

A couple of quick observations for you David.

1) I see you didn’t incorporate the checklist support into the bulk upload screen. Any specific reason for that…even if only as an option? Not a huge issue…As mentioned, that’s the one screen I was able to work out on my own so I can certainly continue to do that if necessary. I’ll just have to be cautious with updates in the future so as to keep the functionality.

2) As to the quick edit area, it looks good except some formatting issues. When using the media assistant, the thumbnail images are initially in a span tag so they are inline. When you open the quick editor, they are in a div so by default, they are block level which pushes the fields down below the image instead of starting next to it. Setting the div to inline or inline-block would avoid that.

3) The checklist columns currently stack on top of each other in the Quick edit mode which looks quite odd and takes up a good bit of vertical real estate (I realize why it’s that way after seeing the bulk edit screen). Since you swap things around a bit for that screen anyway, possibly having any checklist boxes align in a row and make use of any available width for the quick edit screen? Making use of flexbox would make that trivial and avoid any unnecessarily complex CSS to handle it while still making it quite responsive.

You could also likely cap the width of the filename/description area at a certain width, reduce the default size of the checklist boxes and create a good bit of extra horizontal space to avoid having to stack things except on fairly small screens where it’s unavoidable.

Still playing with it, but it should definitely help with edits and updates going forward.

Thanks again for all the assistance you provide.

John

That’s great. I’m putting some finishing touches on that site as we speak so I’ll make a backup and test out the development version today.

Thanks David!

Ahhh great. Much appreciated!

Correct. I tried it out for about 2 days almost 2 weeks ago then disabled it… just hadn’t deleted it yet as I was going to do some additional testing.

Obviously in that time I’ve cleared the cache quite a few times manually aside from the standard purges so can’t see how it would be anything left over cache-wise from that.

well I’ll try adding your code then. It’s odd though as I don’t have either of the lazy loading plugins you list active currently. I do have a3 installed, but it’s not active so not sure why your plugin is trying to lazy load that image.

Odd… according to their site it is exactly what I put in original post.

https://docs.wp-rocket.me/article/15-disabling-lazy-load-on-specific-images

Did they change it or what’s going on?

As I said it the code I used works perfectly when your plugin is off. The code keeps wp-rocket from lazyloading the image.

It’s only when your plugin is active that it doesn’t work. Not only does it ignore it, the image doesn’t display at all.

I’m happy to turn it back on if it’s going to be looked at immediately as it’s obviously not something I can keep on as it affects a large percentage of visitors.

Sure…but I’m not sure how much good it will do you at the moment as I’ve had to turn off the CDN portion of W3T for now due to the issue.

https://simracewaydrivingschool.com

Well that worked like a charm. Thanks!

And thanks for the update about the banner. Still in the early stages so I hadn’t actually tested it while logged out and the banner only is an issue when logged out.

Ok…I’ve run into one issue with how MLA is outputting the tags. When I initially load my test page, it pulls up all the correct images as well as all the correct tags and correct links.

However, I am also using pagination and when I click on to the second page of images, MLA doesn’t produce the tags properly. Instead of still displaying all of the tags for the category, it only displays 2 (and those two aren’t the only tags associated with images on page 2).

If I click the pagination link to go back to page 1 (not the back button in the browser, but the actual “?mla_paginate_current=1” link, the tags are wrong on that page. It seems that anytime the “?mla_paginate_current” is in the URL, it throws off the tags.

Here is the code I’m using for this page.

<div class="galtags">Filter By: [mla_gallery attachment_category='driveways' mla_alt_shortcode=mla_tag_cloud mla_output="list" taxonomy=attachment_tag mla_link_href="{+page_url+}?term_slug={+slug+}&use_tags=yes"]</div>

[mla_gallery attachment_category='{+template:{+request:use_tags+}|driveways+}' mla_alt_shortcode=gallery envira=true envira_gallery_args="pagination=0&tags=0" posts_per_page=15]
[mla_gallery attachment_category='driveways' attachment_tag='{+request:term_slug+}' mla_alt_shortcode=gallery envira=true envira_gallery_args="pagination=0&tags=0" posts_per_page=15]

<div style="text-align:center">[mla_gallery attachment_category='{+template:{+request:use_tags+}|driveways+}' posts_per_page=15 mla_output="paginate_links,show_all" mla_nolink_text='No Gallery Pages']
[mla_gallery attachment_category='driveways' attachment_tag='{+request:term_slug+}' posts_per_page=15 mla_output="paginate_links,show_all" mla_nolink_text='No Gallery Pages']</div>

Here is a link to the example page:

https://www.atdconcretecoatings.com/myportfoliotest/driveways

Any input would be greatly appreciated.

I think that’s going to work just about perfectly. With a bit more tweaking it should be exactly what I need.

Thanks for your assistance!

Ahh…oustanding. I knew there had to be a way.

Absolutely love the plugin by the way.

What I’m wanting to do when the tag is clicked is further filter the gallery

so if the person started out with a gallery full of driveway pictures, then clicking on the tag would filter it down further to the driveway Plus the tag

So say the tags are different type of driveway surfaces such as “acid stain”, “thin pavers”, etc. Then clicking on the “acid stain” tag would then filter to only show driveways that have the acid stain tag as well.

Thanks again for your assistance!