Forum Replies Created

Viewing 11 replies - 1 through 11 (of 11 total)
  • Hi guys,

    From me poking around in the frontend code, this is not a theme issue, it stems back to an incomplete implementation of the img sizes attribute in woocommerce blocks.

    The extent that we experience this depends on what the thumbnail size is set as in our theme relative to the size we try to display the product block. That’s probably the reason it was so quickly classified as a theme issue.

    In my case, the thumbs are defined as being 397px wide, but I am putting the featured product block (with just one product) in a two column layout, so it displays at 609px in the browser.

    The srcset attribute on the image is properly defined, and in my case, actually includes a 610w version of the image!

    But when we take a look at the sizes attribute in the rendered block, it looks like this:

    sizes=”(max-width: 397px) 100vw, 397px”

    This seriously cripples the srcset functionality and displays the thumbnail file no matter the size of the container, thereby producing a blurry image in many use cases.

    I have not delved into the plugin code to find this, but it is easy to remedy by changing the attribute to:

    (max-width: 397px) 100vw, (min-width: 397px) 610px
    or even just
    (max-width: 397px) 100vw, (min-width: 397px) 610px

    Or you could do a full fledged, proper sizes implementation.

    programatically, 397px is the value returned for thumbnail and 610px for full image (as set for single product display in theme).

    Let me know if I need to post this a new ticket to have it remedied in a future release. I see this project has a pretty rapid dev cycle, so I am hopeful that something as easily improved, but with huge implications for users, will get a high priority.

    Also let me know if I can help in any way.

    LJ

    Thread Starter lastafa

    (@lastafa)

    Thanks for your suggestion. I checked the user roles of the users making the untracked orders, and they are set to “customer”, which is not excluded. The only order that is reported on the dashboard is also made by a “customer”.

    I have had a couple of orders placed by returning customers that has not been tracked since my last post. I will keep you updated ??

    Thread Starter lastafa

    (@lastafa)

    Thank you for your swift response Michal!

    Yes, I mean both order and payment. I just had an order registered now after I tweaked all sorts of setting across my site. Will update you if it misses any more or mark as resolved if it continues to work ??

    Looking forward to following your developments on this plugin.

    Thanks,
    Lasse

    I would also like to see this fuctionality. I am soon launching my community, and I’m using your brilliant Kleo theme. Keep up the good work!

    Thread Starter lastafa

    (@lastafa)

    You’re fast!

    This breaks all pages, the source code for any requested page ends with:
    <div id=”message” class=”error”><p>name lookup timed out</p></div>
    <b>Fatal error</b>: Cannot use object of type WP_Error as array in <b>/home/nyta/webapps/ecom/wp-content/plugins/wp-e-commerce-bring-fraktguide/wp-e-commerce-bring-fraktguide.php</b> on line <b>433</b>

    I have to remove it no, site is live ??

    Thread Starter lastafa

    (@lastafa)

    This is the resulting error:

    Fatal error: Call to a member function get_error_message() on a non-object in /home/nyta/webapps/ecom/wp-content/plugins/wp-e-commerce-bring-fraktguide/wp-e-commerce-bring-fraktguide.php on line 430

    Thanks.

    Thread Starter lastafa

    (@lastafa)

    Thanks BODA82, My single.php template was wiped out, thanks for pointing me in the right direction, I was not very resourceful at the time ??

    Forever grateful,
    Lasse

    Thread Starter lastafa

    (@lastafa)

    I upgraded to the final 3.0 and disabled multisite. To my amazement, I could upgrade again after that change to a Norwegian version!

    The norwegian letters are screwed up even in the admin though.. I’m feeling this has a pretty obivious solution. Please patronize me..

    Wow, that worked. My users probably thinks gravatar is a spoof on Avatar: the movie ??

    I’m trying to integrate as seamlessly as possible to get nontech facebook users to start using the social features of my site.

    Is there a possibillity to show the “post to facebook” checkbox for users that already connected their account to facebook or signed up through facebook connect?

    That would really make this swing!

    Thanks

    Thread Starter lastafa

    (@lastafa)

    I changed to utd-8 in wp-config and it helped a lot of the custom fields (had to update the text, but it stuck :))

    The text widget still screws it up though, anyone know why that might be? you can check it out at nyta pre-release sketch.

    Thread Starter lastafa

    (@lastafa)

    Thank you Michael, now I know how to seperate out the individual synergies. Any ideas on how to check for the first post matching the word and return the permalink of that post instead of the synergy page?

    So far I have modified your code to display another link when a post title matches the synergy name, this is real dirty, and does not return correct urls as they have different categories, I’m almost there, need to do a query for the guid and return that instead of the hack.

    <?php
    //if a given taxonomy matches a post name, return the url of that post
      $taxonomy = 'synergi';
      $args=array(
        'orderby' => 'count',
        'order' => 'DESC'
      );
      $terms = get_terms($taxonomy,$args);
      $counter = 0;
      foreach ($terms as $term) {
    
        $counter++;
        if ($counter <= 5) {
    
      global $wpdb;
    if($wpdb->get_row("SELECT post_name FROM wp_posts WHERE post_name = '" . $taxonomy . "'", 'ARRAY_A')) {
    	  echo '<a href="https://nyta.webfactional.com/' . $term->name . '">' . $term->name . '</a>, ';
    	  } else {
          echo '<a href="' . esc_attr(get_term_link($term, $taxonomy)) . '">' . $term->name.'</a> ';
        }
      }
    }
    ?>
Viewing 11 replies - 1 through 11 (of 11 total)