This topic has 18 replies, 3 voices, and was last updated 11 years by Andrei.

  • Author
  • #47369

    Good evening. Is there a suggested logo size to use? I know the baseline starts at 88 pixels high.

    I have a client I am building a website for that has a more vertical logo, so 88 pixels high might be too small.

    Thanks.

    #47507

    Ok, I have discovered a bug / problem. On a new website I am building, I made the logo size 150px tall. It generally works fine if I have the sticky top menu on and resize on scroll on. However, I just turned off the sticky top menu, and even though I have the menu area set at 150px tall, the logo is 88 pixels tall (the default), which makes the logo smaller than desired.

    How do I force the height of the logo to be the size of the image I uploaded (150px in this case)?

    #47714
     Laura
    Moderator

    Hello, you can do it with css, please share a link to your site 🙂

    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
    #47768

    Laura, here is a link to the temp site location, still under serious construction:

    http://c68.bab.myftpupload.com/

    (I can’t publicly show this yet as it needs approval but the customer before it goes live)

    You can see the logo in the top. It is a vertical logo, so it requires some extra height to be more visible. I have the logo uploaded at 150px high and I have the area set for 150px high, so the logo should fill the area (logo has a little spacing within it so it doesn’t bump up to the edge). If I set the header to sticky and resize on scroll, it seems to be fine and fills the area. However, the way I have it set now as not being sticky, the site seems to force the logo to a fixed 88px tall, which causes the area to look odd.

    #48036
     Laura
    Moderator

    Hello, please add this to your quick css or style.css in wp-content/themes/kleo-child/style.css

    COPY CODE
    
    .logo img {
    margin-top: -6px;
    max-height: 100% !important;
    width: auto;
    height: auto;
    }
    
    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
    #48038

    Laura,

    Works perfect! Thanks!

    Just for my reference, is this going to be adjusted in a future Kleo theme update, or will I need to continue to do this across all my websites into the future? I have several websites I will be building that have various height requirements for the logo.

    Thanks!

    #48061
     Laura
    Moderator

    Hello, if you added it to child theme css it will stay there even if you update 🙂

    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
    #48078

    Laura, sorry if I wasn’t clear. I like to keep my child theme css clean, so if Kleo was going to be updated in the future with this coding, I can put a reminder to eventually remove that code … or the opposite, have a reminder in my personal web dev guides to always add it on every install and keep it in there. Just curious. 🙂

    #48122
     Laura
    Moderator

    Hello, when you update the theme, you just have to update kleo and not kleo child, developers should know that

    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
    #48131

    Laura, I do understand that as I am a web dev and have spent quite a lot of money on themes over the years, including multiple licenses of Kleo (and still plan on buying more for future web projects). This is something I do know, and my apologizes if you think I am a clueless developer.

    I think you might be completely misunderstanding what I am asking here.

    Kleo main theme has undergone several theme updates since it was launched, fixing various bugs and adding various new features and plugin adaptations. To me, I consider this situation a bug or a flaw in the programming that intentionally restricts, by the coding, the size of the logo. Nowhere in the documentation that I read or saw (maybe I missed it) that stock Kleo forces and only uses logos 88px tall. So, again, this to me is a bug, and the adjustment you provided would be added into the stock programming in a future Kleo update.

    If that is the case, when that happens, I can then remove that coding from my Kleo Child Theme coding as it would be unnecessary to have. In other words, as I mentioned before, I would then remove it to keep it clean so I do not have unnecessary coding in there.

    So my question is, do you all find this as a bug and will it be updated in a future release, or is this not considered a bug and will not be altered in the future?

    That is all I was asking. Just a friendly check to see if you all classified this as a bug with a temp fix for me that will be applied to a future Kleo release. I certainly didn’t intend on this going this far or somewhat be considered a clueless developer.

    #48193
     Andrei
    Moderator

    Hi Mate,

    Could you please give me admin credentials to play a little bit with the floating menu, and see if there are any bugs ?

    Looking forward.

    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
    #48204

    No prob Andrei … here is the info:

    site: http://c68.bab.myftpupload.com/
    admin: http://c68.bab.myftpupload.com/wp_admin
    U: Kleo
    P: asoiff255!?!safs

    This is not a “live” website, and I am not planning on working on it more until Monday afternoon, so play around all you would like.

    #49293
     Andrei
    Moderator

    Ok, so I see you managed to figure out from where to increase the placeholder of the logo, which is the “Main menu height” setting. From what I’ve tested it works just find even without the code that Laura given you.

    Hope this is clear now.
    Let me know if I can help you with anything else.
    Cheers

    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
    #49296

    Andrei, I do not see it working without code on my end. I just removed that coding, and the logo shrunk immediately. If I add the code back in, the logo is correct size.

    I have attached two screenshots (large is with code, small is without code).

    I currently have the code removed on the site version if you want to check on your end.

    #49302
     Andrei
    Moderator

    I just tested once more on 3 different browsers and in all of them i see the big version of the logo. Could you please try and clear you cache or try from a different browser ?

    Cheers

    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
    #49316

    I cleared my cache and still get the smaller logo rather than the large. This is on Chrome browser (my primary). I tried on IE and got the same. I also tried on the Sony PlayStation 4 browser and got the same smaller logo versus the logo that fills entire area.

    I attached an image I shot towards my TV so you can see what it looks like on PS4 browser.

    #49324
     Andrei
    Moderator

    Did some more tests, and it seems that this is a nasty small bug which happens just under Windows environment, and I was testing from a OSX system, from where it works just fine.
    We’ll get more deeper into this and see what’s the issue, and we’ll probably fix it in our next update.
    Meanwhile you can keep the code that Laura given you.

    Thanks for your collaborations.
    Cheers

    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
    #49325

    Good deal… glad I could help at least narrow it down a bit. I love this theme … already purchased six licenses for six websites I have developed, and have three more websites in the works over the next week or two (and more current client conversions coming down the line!).

    #49328
     Andrei
    Moderator

    I’m glad to hear that, good luck with the business.

    Cheers

    Hi there!!! Help others from the community and mark any reply as solution if it solved your question. Mark as a solution
Viewing 19 posts - 1 through 19 (of 19 total)

The forum ‘KLEO’ is closed to new topics and replies.

Log in with your credentials

Forgot your details?