• Resolved kense

    (@kense20xx)


    Hi Anders,

    Just a quick one. Since the latest Gutenberg update, there seems to be an issue with at least one of your pre-built Tove Header patterns.

    When using the ‘Header with opening hours, site title, social icons, a menu and a call us button’ and replacing the Site Title with the Site Logo, the logo has a strange alignment issue.

    Your blocks.css file adds a property as follows:

    .wp-block-site-logo.aligncenter {
        text-align: center;
    }

    However, the logo is also being affected by some inline css that reads:

    .wp-block-site-logo.aligncenter {
        display: table;
    }

    I’m presently hunting down the source of that inline css to add a space between .wp-block-site-logo and .aligncenter because that seems to fix the issue.

    I am assuming it is something new in Gutenberg, as your pre-built pattern worked fine before. I just wanted to flag it here in case anyone else was having the same issue or it was something you were unaware of.

    Tove version 0.4
    Gutenberg version 11.8.0

Viewing 3 replies - 1 through 3 (of 3 total)
  • Theme Author Anders Norén

    (@anlino)

    Hi @kense20xx,

    Thanks for the heads up – I’m not sure what the reasoning for that CSS is in Gutenberg, since display: table doesn’t seem to do anything to center the logo, but I’ve fixed it in Tove itself for now. It’s included in version 0.4.1, which should be available as an update.

    — Anders

    Thread Starter kense

    (@kense20xx)

    Thanks Anders. I saw the update and appreciate the shout out in the readme file, but it seems that the display: table issue is still present. I’ll open a thread in the Gutenberg plugin forum and see if anyone there can fix it.

    Thread Starter kense

    (@kense20xx)

    In fact, ignore my last message. I see you’ve added a display: block to the variable, it must be a caching issue at my end.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Issue with wp-block-site-logo in Tove header pattern’ is closed to new replies.