Support » Theme: Tiny Forge » How can I display tall featured images in header?

  • Resolved WebTrooper

    (@webtrooper)


    First let me say Thank You for this excellent theme… it’s exactly what I need for my project. Well, almost. ๐Ÿ™‚

    One of my favorite features of Tiny Framework is the way it uses featured images in the header when visiting the respective article. However, it will only work if the image is wider than it is tall, or basically only works for landscape layout images. If an image is even 1 pixel taller than it’s width, or portrait layout, it will not display in the custom header.

    Looking at the code in functions.php, I see the height limit is set to 9999 (i.e. unlimited), so it seems it should show portrait layout images as well. But it doesn’t. How can I fix this?

    Thanks

Viewing 12 replies - 1 through 12 (of 12 total)
  • Theme Author Tomas Mackevicius

    (@tomasm)

    Hi, first if it is new project I recommend using new version of this theme, please see here:

    But looks like you already using it ๐Ÿ™‚

    The theme looks if image is equal or bigger than 960px and only then displays it as a header image.

    So you have several options:

    – you can size the image so its width is 960px or more.

    – you can open header.php and try to comment out or delete this part of the code:

    && $image[1] >= $header_image_width

    it’s in lines 145 and 146.

    If image is smaller than 960px it might look not very good, but you will see if it is good for you.

    And share your site to look at the results ๐Ÿ˜‰

    Thread Starter WebTrooper

    (@webtrooper)

    Thanks for the quick reply. It is a new project and I thought I was using the latest version of Tiny Framework, but turns out I wasn’t. I downloaded and installed from the link you provided, but it seems there are some issues with that version — the nice search box does not appear in the menu, and the responsive menu does not display.

    Oddly though, the search box does appear in the child theme. But the responsive menu is not present in either child or parent theme… the main menu just disappears at 770px screen size.

    Houston, we have a glitch! ๐Ÿ˜ฎ

    UPDATE: I finally got a decent signal on my phone, so was able to test it on an actual small device… and everything works fine with both parent and child theme. The search still doesn’t appear using my desktop (windows 7, chrome, ff and ie). I’ll just be using child themes so I guess it’s not a big issue for me, but something you may want to look into.

    Theme Author Tomas Mackevicius

    (@tomasm)

    That’s correct, search in the top menu is not included in main theme – those who dare to use child themes should get some kind of reward ๐Ÿ™‚

    Also please try to reload site on desktop with CTRL+F5 to clear the cache, see it will solve the issues.

    Thread Starter WebTrooper

    (@webtrooper)

    When I refreshed the page I got a quick glimpse before the admin bar loaded… the responsive menu was hiding underneath it. ๐Ÿ™‚

    With admin bar

    Without admin bar

    Theme Author Tomas Mackevicius

    (@tomasm)

    WebTrooper, thanks for sniping down that bug! ๐Ÿ˜‰

    You see, first thing I do on a new site – disable admin bar, that’s why I was not aware of it… but thanks to you, buddy, it is fixed now:

    BTW I checked your site, do you want to achieve something like:

    chicagomuncorp.org
    exportautousa.com

    Because I was planning to start selling that child theme for 25. You can be one the first devs to get it, just donate that amount.

    Thread Starter WebTrooper

    (@webtrooper)

    Well, I’m glad I could help! ๐Ÿ˜€

    Are you referring to the sub-site, bubcity.com/woody? I see the similarity with exportautousa.com but what I have in mind is very different. Once I get the general layout, my intention is to make copies of the child theme with different backgrounds, colors, fonts and such, then provide some further options through the front-end customizer. Bub City is to become a blogging community, and these will be the themes users can choose from. Hopefully, I’ll have the overall layout ready today.

    As for making a contribution for your efforts, I already plan to as soon as I get paid, probably middle of this week. I’ve tried literally close to 300 themes, both free and paid for, and I kept coming back to tiny framework. Considering how much time and money I’ve already spent trying to find the right theme, and how much work you’ve obviously put into creating this one, I think $25 is not nearly enough.

    I also plan on crediting you for providing the theme, just as I plan on crediting Arisath for providing Kirki, Aautomatic for providing WordPress, and providers of numerous other resources I will be using.

    Anyway, about the featured images as used for the header image ๐Ÿ˜€
    I understand that smaller images won’t work very well, especially with full width headers. What you suggested in your original replay, to delete or comment out the line of code will make all featured images appear in the header, even the tiniest ones. And actually, I already tried that before posting my inquiry :). I’m happy with keeping the 960 minimum width, but the issue is that portrait layout images are filtered out. So if the featured image is 960 wide by 961 tall, or any image taller than it is wide, it will not be used as a header image.

    The image is not a big issue, though. If only landscape layout images are displayed in the header, it’s still a pretty cool feature. It would just be nice to have the option to use portrait images as well.

    Theme Author Tomas Mackevicius

    (@tomasm)

    Interesting, I never tried that combination (height > width). Will try to experiment with that.

    And thank you very much for the encouraging words!

    Thread Starter WebTrooper

    (@webtrooper)

    Hi Tomas, hope you’re doing well. I just want to let you know I contributed $65 today, to help further the development of this awesome theme. I trust it will come in handy. Thanks for all your help! ๐Ÿ™‚

    – Ted

    Theme Author Tomas Mackevicius

    (@tomasm)

    Thank you!!!

    Theme Author Tomas Mackevicius

    (@tomasm)

    I sent email to your PPal address. I hope you received it.

    Thread Starter WebTrooper

    (@webtrooper)

    Thanks for reminding me, I have to update my PP email. ๐Ÿ™‚

    I sent you an email from the proper one to reach me. …@msn.com

    Thread Starter WebTrooper

    (@webtrooper)

    Thank you! ๐Ÿ˜€ ๐Ÿ˜€

Viewing 12 replies - 1 through 12 (of 12 total)
  • The topic ‘How can I display tall featured images in header?’ is closed to new replies.