[calebweeks]

Thanks, Ed!

Did the fix make it to the Theme Version: 1.3.12 by chance?

No worries if not!

Thanks again for the short-term fix.

[calebweeks]

Hi Swift Team,

Question 1: Would you all be willing to explain a little more of what this means?

Question 2: Are you saying that if we use WooCommerce and “Force HTTPS on checkout” (or the My-Account Page, etc.) that we need to turn of the Swift Smart Script?

Question 3: Is there a way where you guys could update it so that it could work with HTTPS or is that too hard?

We would love to use the Smart Script but we definitely need HTTPS for checkout/credit card payments, etc.

Thanks guys!

[calebweeks]

(And I fixed it temporarily with this:

/*WooCommerce Quickview - mobile*/
@media (max-width: 1020px) {
	#jckqv #jckqv_summary, #jckqv #jckqv_images_wrap {
		width: 100%;
	}
}

[calebweeks]

Hi again guys – just an update that I think it is a simple CSS mistake.

Currently the desktop styles for #jckqv_images_wrap have it with a width: 50%; and then the same #jckqv_summary at 50%. Those rules should target only the Quickview on desktop using @media queries, etc. i.e. @media (min-width: 1020px) {#jckqv_images_wrap {width: 100%;}}

I think that would protect the already included CSS that is targeting the mobile version of the Quickview, etc.

I hope that makes sense! 🙂

Thanks guys!

[calebweeks]

Great – I’ll anticipate it in the update. Thanks, Kyle!

[calebweeks]

Hi again Swift Team,

Would you have any insight, David, on how to do this in the jQuery with only the desktop?

The PreOrder Button on the image hover doesn’t seem to work on Mobile which makes me think we’ll have to load each item (individual product page) for mobile visitors – would you guys be able to confirm?

Thanks again for all your help and I definitely realize this is out of the scope of normal theme support so if you need to decline, I completely understand. Thanks guys!

[calebweeks]

Thanks, Ed!

Nice work figuring that out.

Will look for it in a future update.

Thanks again to @erikbalster for the CSS in the meantime! 🙂

[calebweeks]

Sure thing, Ed: christynockels.com/christmas

Would you need/want WP-Admin access?

Thanks guys!

[calebweeks]

Okay – it’s working (your jQuery code was so easy to read/understand with the comments – thank you for doing that!)

The one adjustmnet I made was to just target the image link itslef and leave the others active.

That was on Line 4 where I specified the class (in this case product-link) for the <a> tag.

Line 4 then became: jQuery("ul.products li.product a.product-link").removeAttr('href');

Thank you again, David, for taking the time to help – it helped so much!

[calebweeks]

Actually, David, it had one quirk: it also disabled the “Product Title” (underneath the image) from linking to the post.

In a perfect world, clicking on the image/thumbnail would launch Quickview but clicking on the Title would still allow the visitor to visit the actual product page.

I am looking at the code but wanted you to know in case my previous enthusiastic “that did the trick!” message reaches you quickly.

If this isn’t possible, I understand, and thank you again either way!

[calebweeks]

Thanks for the reply, David – that totally worked!

I swapped .hasClass("home") with the Page ID (in my case .hasClass("page-id-890") and it totally worked.

Thank you guys so much!

[calebweeks]

Thanks so much, Kyle!

Sorry to bug with you that as it turned out to be much more simple than I thought.

Final Question: Do you think Uplift is supposed to have this by default or was it custom code that you guys added for the demo?

It was kind of strange to not to have it behave properly. If it was/is a bug, glad to bring it to someone’s attention and if not, thanks for the code!

[calebweeks]

Hi Swift Team,

Question: Would you all be willing to update this in the next theme update?

Currently, for whatever reason here in July, this appears to still be an issue, though the Custom CSS @erikbalster offered definitely did the trick!

Would save us the searching here on the forums. Thanks for considering!

P.S. I am using Uplift 1.2.60

[calebweeks]

Ah, perfect – that makes sense.

Thanks again, Kyle for your quick and helpful replies! I’ll do my best not to ask too many questions, but when I do, I really appreciate you being there (as I’m sure many customers do, too)!

Onward. 🙂

[calebweeks]

Hey guys – scratch that!

I see an option in the Page Meta > “Extra Paragraph Spacing” that accidentally was checked.

Can safely ignore this ticket!

P.S. What use case would something like extra paragraph spacing be good for? No obligation to reply. Just curious… Thanks again!