• Resolved roadsunknown

    (@roadsunknown)


    On the Kinetic Posts block, the CSS is being applied to the parent <h5> tag and not the child <a> tag. For themes where the <a> is stylized (most), the block settings are ignored.

    If Post Title Font Size is blank:

    .wp-block-kioken-kinetic-posts .grid-post .grid-post-row .grid-content-area h5.grid-content-title {
        font-size: 24px;
        line-height: 1.5;
        font-weight: bold;
        margin: 0;
        margin-bottom: .3em;
    }

    If Post Title Font Size is configured, this CSS overrides the above:
    #kt-kinetic-posts__23b72c .grid-post-row h5.grid-content-title {font-size: {{setting}}px; }

    The effect here is the font-size, font-weight, etc. are not applied as there are more specific classes on the Heading <a> tags thus in my case h5 a {font-size: 16px;} is rendered (see page link to debug).

    If you add “a” at the end of the class declaration, i.e. .wp-block-kioken-kinetic-posts .grid-post .grid-post-row .grid-content-area h5.grid-content-title a { ... } the styles are applied correctly.

    Also, it would best to allow the theme to control the default font-size, font-weight, line height but also allow the ability to override (no setting = inherit, setting = override). Related issue, my global headings use font-weight: 600 but Kinetic Posts block is overriding with generic “bold”.

    Furthermore, the block does not allow setting the <h> level to render (using H5 as default). Similar blocks (competitors) allow selection of H1, H2, H3, H4, H5 or H6. This is important for best practices page design.

    Really liking the plugin though and hope to continue using more. Thanks!

    • This topic was modified 1 month ago by  roadsunknown.
    • This topic was modified 1 month ago by  roadsunknown. Reason: Formatting
    • This topic was modified 1 month ago by  roadsunknown. Reason: Formatting
    • This topic was modified 1 month ago by  roadsunknown. Reason: Formatting

    The page I need help with: [log in to see the link]

Viewing 1 replies (of 1 total)
  • Plugin Author onurozt

    (@sj_o)

    Thank you for the report on this Brendon.

    We’re not super happy with the semantics of the Kinetic Posts’ markup as well. I’ll probably refactor the whole kinetic posts output markup, fixing your current and previous issue this weekend.

    Also making the posts having schema.org markup as well.

    Will let you know.

    Thanks!

Viewing 1 replies (of 1 total)
  • You must be logged in to reply to this topic.