Home Forums Newsletter Plugin Support Wrong code in responsive Theme

Viewing 4 posts - 1 through 4 (of 4 total)
  • Author
    Posts
  • #29436
    Rico
    Participant

    Hi Stefano,

    is there a bug in wp-content\plugins\newsletter\emails\tnp-composer\blocks\header-01-header.block.php
    <?php echo isset($block_options['header_logo']) ? $block_options['header_logo']['url'] : 'http://placehold.it/180x50?text=' . isset($block_options['header_title']) ? $block_options['header_title'] : "" ?>
    I think it must be
    <?php echo isset($block_options['header_logo']) ? $block_options['header_logo']['url'] : 'http://placehold.it/180x50?text=' . isset($block_options['header_title']) ?>

    Greetings,
    Farant

    #29438
    Rico
    Participant

    Another problem in wp-content\plugins\newsletter\emails\tnp-composer\blocks\content-06-posts.block.php:
    <td valign="top" style="padding: 40px 0 0 0;" class="mobile-hide tnpc-row-edit" data-type="image">
    but your CSS Selector is
    td[class="mobile-hide"]{ display:none;}
    maybe better:
    <td valign="top" style="padding: 40px 0 0 0;" class="mobile-hide"><div class="tnpc-row-edit" data-type="image"> ... </div>

    #29439
    Rico
    Participant

    Me again 🙂
    In wp-content\plugins\newsletter\emails\tnp-composer\blocks\content-07-twocols.block.php and wp-content\plugins\newsletter\emails\tnp-composer\blocks\content-06-posts.block.php:
    If there is no post image tnp_post_thumbnail_src return only http: and the tag looks like <img src="http:" ...

    #29524
    Rico
    Participant

    There are a few http connects like http://placehold.it... and http://cdn.thenewsletterplugin.com....
    If we use the Plugin in a https area we get some errors (in the developer console).

Viewing 4 posts - 1 through 4 (of 4 total)
  • You must be logged in to reply to this topic.