Kirki: Standard Fonts in Typography field throw an error

[hellor0bot]

Issue description:

When using Standard Fonts and the output parameter, we're getting the error:

Notice: Array to string conversion in /kirki/modules/css/field/class-kirki-output-field-typography.php on line 65

Generated CSS:

font-family: Array;

Version used:

develop branch

[aristath]

Can you post the code you're using?
I'll deal with all typography issues this weekend so having some code to test against would make things easier 👍

[hellor0bot]

Sure!

But there's really nothing special:

Kirki::add_field( 'theme_name_theme_mod', array(
    'type'        => 'typography',
    'settings'    => 'typography_headings',
    'label'       => esc_html__( 'Heading', 'text-domain' ),
    'section'     => 'typography_headings',
    'default'     => array(
        'font-family'    => 'Montserrat',
        'variant'        => '600',
        'subsets'        => array( 'latin-ext' ),
        ),
    'priority'    => 10,
    'output'      => array(
        array(
            'element' => 'h1, h2, h3, h4, h5, h6',
            ),
        ),
) );

[vlthemes]

Same mistake

[aristath]

Fixed.
Thanks for reporting this!

[hellor0bot]

I've just tested it, and seems there's an issue with switching font-weight for standard fonts. The weight doesn't get changed :(

[hellor0bot]

For reproducing the issue, try changing between weights in Serif, then switch to Sans-Serif and change font-weights.

[aristath]

@hellor0bot I just tested it and can't replicate the issue you're describing

[hellor0bot]

I've just added transport: auto to all typography fields and removed line breaks in the element parameter and it's working fine now.

By the way, transport: auto on typography fields is so awesome. Great job!

[hellor0bot]

Nope, the issue is still there and I can't debug it :(

All fonts appear in the Customizer correctly with transport set to auto, however after saving a former (or incorrect) variant is displayed on the front-end.

[hellor0bot]

The issue is related to all fonts, not standard ones, as I'd thought before.

[aristath]

Should be ok now.

[hellor0bot]

Just checked it. All is good. Thanks a lot!