Forum Replies Created

Viewing 8 replies - 1 through 8 (of 8 total)
  • Thread Starter charlieroark

    (@charlieroark)

    Thanks for your reply Shawn,

    I was told that there were still instances of the old name showing up places, but I’m not seeing them now, and it appears that even if I was, it wouldn’t effect peoples’ feeds. Thanks!

    Thread Starter charlieroark

    (@charlieroark)

    Yeah, I don’t know who was hired to make the patch, it was before I started working here months ago, but for whatever reason it didn’t work with the simple changes. The patch also customized the look of the sheets beyond just making the links work, but that’s besides the point. I’ll be employing your <a href suggestions but wanted to look for an alternative to editing functions first since you explicitly state that it’s not the best solution.

    Thanks for your response. I will try to figure it out from what others have written.

    Thread Starter charlieroark

    (@charlieroark)

    Hi Midatan,

    Thanks for all of the help. I added the code to my enable-sticky-header.js file but it did not appear to do anything. I tried both of the code above with no go. Here’s what my file looks like:

    /**
     * Theme Enable Sticky Navigation
     * @author Pitabas Behera
     */
    jQuery(document).ready(function() {
        function ascentStickyHeader() {
            if (jQuery(window).scrollTop() >= origOffsetY) {
                jQuery('#header-main').addClass('sticky');
                jQuery('#header-main.sticky').css({
                    'top': adminbarHeight + 'px'
                });
    
                var headerSearchFormTopPosition = (adminbarHeight) + (jQuery('.header-bottom').outerHeight());
                jQuery('#header-search-form-wrap').addClass('sticky');
                jQuery('#header-search-form-wrap.sticky').css({
                    'top': headerSearchFormTopPosition + 'px'
                });
            } else {
                jQuery('#header-main').removeClass('sticky');
                jQuery('#header-main').css({
                    'top': 'auto'
                });
                jQuery('#header-search-form-wrap').removeClass('sticky');
            }
        }
    
        function wpAdminbarHeight(wpAdminbar) {
            if (jQuery(wpAdminbar).length) {
                var adminbarHeight = jQuery(wpAdminbar).outerHeight(true);
            } else {
                var adminbarHeight = 0;
            }
            return adminbarHeight;
        }
        jQuery(window).on("load resize scroll", function(e) {
            adminbarHeight = wpAdminbarHeight("#wpadminbar");
            if(jQuery(window).innerWidth() > 767) {
                jQuery('#header-main.sticky').css({
                    'top': adminbarHeight + 'px'
                });
            } else {
                jQuery('#header-main.sticky').css({
                    'top': 0
                });
            }
    
            var headerSearchFormTopPosition = (adminbarHeight) + (jQuery('.header-bottom').outerHeight());
            jQuery('#header-search-form-wrap.sticky').css({
                'top': headerSearchFormTopPosition + 'px'
            });
        });
        adminbarHeight = wpAdminbarHeight("#wpadminbar");
        var stickyMenu = jQuery('#header-main');
        var origOffsetY = (stickyMenu.offset().top) - (adminbarHeight);
        document.onscroll = ascentStickyHeader;
    
    	jQuery("#header-main").wrap('<div class="header-platzhalter")></div>'); //Placeholder is created
    	jQuery(".header-platzhalter").height(jQuery("#header-main.header-bottom").outerHeight()); //Placeholder is set to the height of the header
    
    window.onresize = function() {
        jQuery(".header-platzhalter").height(jQuery("#header-main.header-bottom").outerHeight());
    
    }
    
    }); //End jQuery
    

    Thanks!

    Thread Starter charlieroark

    (@charlieroark)

    Closing and marking this topic as “resolved”.

    Thread Starter charlieroark

    (@charlieroark)

    Everything seems to be alright now. The issue was that with the new update to the plugin, ad code wasn’t being recognized or rendering. The biggest issue was the bottom widget in Block 1 not rendering at all with the old settings, but under these new settings it displays. DFP wasn’t displaying at all, even under the exact same settings as the Adsense units, so I’m not sure why DFP is suddenly working. I’m just glad that it is.

    Thanks!

    Thread Starter charlieroark

    (@charlieroark)

    Hi there,

    Thanks so much for your prompt response, I didn’t see your earlier reply.

    Yes, I am, and we have more blocks enabled than just that. We also have block 1, 2, and 4 running.

    Block 1 is supposed to be an ad widget after the article and above the author section. I just changed it to have the same settings as Block 9 and it is now displaying, but it wouldn’t display with any other settings.

    Block 2 is an ad unit for a company called Lanista. We only put this one into select articles, and normally code for it with shortcode. This is working fine.

    Block 4 also appears to be working just fine at the moment, and is running Adsense code (originally it was DFP code but this was not displaying at all – I will try replacing it with the DFP code and let you know if it fails to display).

    Thread Starter charlieroark

    (@charlieroark)

    I’ve since deleted this plug in and given it a one star review. I’m not sure if it’s effective on low volume websites, but it certainly wasn’t for a website generating a lot of traffic. I won’t be downloading it again but if there is an answer, I’ll leave this topic open so someone else can assist future users. Thanks.

    Thread Starter charlieroark

    (@charlieroark)

    Sorry, I figured out the issue. I just need to add the following line of CSS:

    #ad-inserter > ul {
    list-style-type: none;
    padding: 0;
    }

    Thanks!

Viewing 8 replies - 1 through 8 (of 8 total)