Related Products 1.5

[dharrison]

Hi mc

What's the value of SHOW_PRODUCT_INFO_COLUMNS_FEATURED_PRODUCTS?

3 which is the default. so is the SHOW_PRODUCT_INFO_COLUMNS_RELATED_PRODUCTS

For example, the following is specifically causing this plus any others that might be on the page:
Code:

<script src="http://cdnjs.cloudflare.com/ajax/libs/jquery-easing/1.3/jquery.easing.min.js" type="text/javascript"></script>

OK I have changed this to https://. But only really saw this as a potential issue. http is referencing other sites in a few cases of my code but I cannot see how this would have any affect on the rest of my code. Plus not all websites are https:// yet.

I have tried the code again and using https://shaftdirect.com/office-seati...air-p-110.html as an example, it hasn't worked.

[mc12345678]

Thanks mc

I tried that

// show only when 1 or more
if ($num_products_count > 0) {
  if ($num_products_count < SHOW_PRODUCT_INFO_COLUMNS_FEATURED_PRODUCTS || SHOW_PRODUCT_INFO_COLUMNS_FEATURED_PRODUCTS == 0) {
    $col_width = floor(100/$num_products_count);

if ($num_products_count == 2) {
  $col_width = 49;
}
    
  } else {
    $col_width = floor(100/SHOW_PRODUCT_INFO_COLUMNS_FEATURED_PRODUCTS);
  }
  while (!$featured_products->EOF) {

But no joy.

When you say "Mixed content" what do you mean? I haven't seen that

There is still something "not right" here... I am looking at the code that is provided with the module and looking at the result on the site. With the above suggested code, given that SHOW_PRODUCT_INFO_COLUMNS_FEATURED_PRODUCTS == 3, $num_products_count == 2, this means that the first section of the if statement is expected to be followed and therefore initially $col_width would be set to 50. But, then because $num_products_count == 2, the next inner code is expected to be executed which sets $col_width == 49. There is no code that follows modifying $col_width therefore when the html is assembled for the product the width:49% should be applied.

This brings me back to the question of if includes/modules/winchester_responsive/related_products.php exists. If it does, then the change recommended above about using the if ($num_products_count == 2) … should be applied to that file. Otherwise, verify that the file has been saved and/or that it is not write protected in trying to save it.

Hi mc



3 which is the default. so is the SHOW_PRODUCT_INFO_COLUMNS_RELATED_PRODUCTS



OK I have changed this to https://. But only really saw this as a potential issue. http is referencing other sites in a few cases of my code but I cannot see how this would have any affect on the rest of my code. Plus not all websites are https:// yet.

I have tried the code again and using https://shaftdirect.com/office-seati...air-p-110.html as an example, it hasn't worked.

There are some "nuances" about what can/can not begin with http: For example a straight out clickable link, depending on the browser, could begin with http:, but an image or a script's "src=" should not be different than the presented page. Some ways around the "concern" of one or the other includes using no prefix and just beginning with the double forward slash followed by the domain name and remainder of the address, otherwise to use say $request_type to identify whether it should begin with https: or http: respectfully (if $request_type == 'SSL' then https: should be used otherwise http:).

Anyways, I also now see that there are 2 console issues identified when using Microsoft Edge and that there appear to be four unmatched end tags which could play a factor in how things are displayed (ie. could be the cause of your problem and all this "math" stuff may not be necessary.

Suggest running the page through an HTML validator in some way or fashion to be sure that the template and page are formatted correctly and then work on the display issues.

[raptar]

Try changing

.centerBoxContentsRelatedProduct { display: inline-block;}

to

.centerBoxContentsRelatedProduct {display: inline-table;}