Other bugs in Complete Pro

Home Forums All Other Themes Other bugs in Complete Pro

Tagged: ,

Viewing 10 posts - 1 through 10 (of 10 total)
  • Author
    Posts
  • #45464
    Michael Como
    Member
    • Topics: 2
    • Replies: 5
    • Total: 7
    Member since: August 25, 2016

    Missing file: assets/js/jquery.easing.js

    complete.js?ver=1:184 Uncaught TypeError: Cannot read property ‘classList’ of null

    wp-emoji-release.min.js?ver=4.6:2 Uncaught SyntaxError: Unexpected string

    #45486
    SKT Themes
    Keymaster
    • Topics: 8
    • Replies: 6705
    • Total: 6713
    Member since: June 12, 2013

    Can you explain in which browser or your website URL reference can be given where its shown as missing so that we can understand and rectify it.

    Regards,
    Shri

    #45496
    Michael Como
    Member
    • Topics: 2
    • Replies: 5
    • Total: 7
    Member since: August 25, 2016

    Sure, it’s here: http://www.inspectcheckauto.com/
    On latest Chrome Win 10.

    The first error is actually a file that doesn’t exist in the actual theme zip file.

    On latest Firefox Ubuntu I’m getting just:
    TypeError: document.querySelector(…) is null
    complete.js:184:39

    which matches the line of the 2nd error above.

    #45537
    SKT Themes
    Keymaster
    • Topics: 8
    • Replies: 6705
    • Total: 6713
    Member since: June 12, 2013

    Hi Michael,

    Our coders said in the demo version and the theme we have doesn’t have this bug so which plugin causes such an error is what we wanted to test.

    Would it be possible for you to give us admin details? If so send us the same via sktthemes.net/contact

    Regards,
    Shri

    #45538
    SKT Themes
    Keymaster
    • Topics: 8
    • Replies: 6705
    • Total: 6713
    Member since: June 12, 2013

    Okay i further understood the problem and you have removed twitter from footer.

    Whenever you want to replace footer widgets here is a video which will help you: How to set up own widgets and remove default footer widgets: https://www.youtube.com/watch?v=4a1cR34OGMU

    Regards,
    Shri

    #45544
    Michael Como
    Member
    • Topics: 2
    • Replies: 5
    • Total: 7
    Member since: August 25, 2016

    Are you saying we can’t remove Twitter? We changed the Footer Layout under Footer Style to 2 Columns. Now I’ve changed all the footer widgets to text and nothing has changed as far as errors.

    #45882
    Michael Como
    Member
    • Topics: 2
    • Replies: 5
    • Total: 7
    Member since: August 25, 2016

    Also, the CSS for the Services boxes isn’t coded properly.

    At Desktop if one box is the removed the remaining three aren’t centred.
    At Mobile if there’s only a small amount of text in a box it doesn’t match the width of the other boxes.
    http://www.inspectcheckauto.com/

    #46023
    SKT Themes
    Keymaster
    • Topics: 8
    • Replies: 6705
    • Total: 6713
    Member since: June 12, 2013

    Hi Michael,

    There is no css issue. currently for services are you using following shortcode :

    [service pattern=”boxpattern-1″ bgcolor=”#81cccd” icon=”http://www.yoursite.com/wp-content/themes/complete/images/wordpress.png” title=”Worpress” button=”Read” url=”#”]Lorem ipsum dolor amet consectetur adipiscing elit. Nunc mi dui placerat eu faucibus vel, efficitur quis magna. Pellentesque habitant morbi tristique senectus et netus et malesuada.[/service]

    So if you want to show

    Four columns in a row use : pattern=”boxpattern-1″

    Three columns in a row use : pattern=”boxpattern-2″

    Two columns in a row use : pattern=”boxpattern-3″

    Single columns in a row use : pattern=”boxpattern-4″

    For Current situation if you want to make your service box center so please change pattern=”boxpattern-1″ to pattern=”boxpattern-2″ its solve your problem.

    For twitter in case if that is a difficulty we can remove the whole Twitter code from our theme and update the theme.

    Regards,
    Shri

    #46072
    Michael Como
    Member
    • Topics: 2
    • Replies: 5
    • Total: 7
    Member since: August 25, 2016

    Hi Shri

    Yes, I can use boxpattern-2, although it’s not ideal. This doesn’t do anything about how ugly the boxes look at mobile, so I’ve fixed that one for you.

    @media screen and (max-width: 29.938em)
    should have “width: 100%”
    for .boxpattern-1, .boxpattern-2, .boxpattern-3, .boxpattern-4
    not “auto”.

    As for Twitter, I’m just highlighting bugs for fixing. But If something simple in JS like wrapping DOM accesses in IF checks is too hard then I suppose I shouldn’t bother.

    #46224
    SKT Themes
    Keymaster
    • Topics: 8
    • Replies: 6705
    • Total: 6713
    Member since: June 12, 2013

    We have updated the theme with the twitter feed removed now and it is a widget so one can add anything they need.

    Regards,
    Shri

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