2.2 regression: Can't leave theme-color unset

K

Kirby

Well-known member
Licensed customer
Affected version
2.3.10
XenForo 2.2 PAGE_CONTAINER had the following code

Code: 
<xf:if is="property('metaThemeColor')">
    <meta name="theme-color" content="{{ parse_less_color(property('metaThemeColor')) }}" />
</xf:if>

So if style property metaThemeColor was empty no meta tag was generated at all so the browser used its defaults.

In XenForo 2.3 the code has changed to
Code: 
<xf:if is="$xf.style.isVariationsEnabled()">
    <xf:if is="$xf.visitor.style_variation">
        <meta name="theme-color" content="{{ parse_less_color(property_variation('metaThemeColor', $xf.visitor.style_variation)) }}" />
    <xf:else />
        <xf:if is="$xf.style.hasAlternateStyleTypeVariation()">
            <meta name="theme-color" media="(prefers-color-scheme: {{ $xf.style.getDefaultStyleType() }})" content="{{ parse_less_color(property_variation('metaThemeColor', 'default')) }}" />
            <meta name="theme-color" media="(prefers-color-scheme: {{ $xf.style.getAlternateStyleType() }})" content="{{ parse_less_color(property_variation('metaThemeColor', $xf.style.getAlternateStyleTypeVariation())) }}" />
        <xf:else />
            <meta name="theme-color" content="{{ parse_less_color(property_variation('metaThemeColor', 'default')) }}" />
        </xf:if>
    </xf:if>
<xf:else />
    <meta name="theme-color" content="{{ parse_less_color(property_variation('metaThemeColor', 'default')) }}" />
</xf:if>

With this code it is no longer possible to not generate the meta tag, eg. a theme color is always set and it is not possible to use browser default.

Possible Workaround
Code: 
<xf:if is="$xf.style.isVariationsEnabled()">
    <xf:if is="$xf.visitor.style_variation">
        <xf:if is="property_variation('metaThemeColor', $xf.visitor.style_variation)">
            <meta name="theme-color" content="{{ parse_less_color(property_variation('metaThemeColor', $xf.visitor.style_variation)) }}" />
        </xf:if>
    <xf:else />
        <xf:if is="$xf.style.hasAlternateStyleTypeVariation()">
            <xf:if is="property_variation('metaThemeColor', 'default')">
                <meta name="theme-color" media="(prefers-color-scheme: {{ $xf.style.getDefaultStyleType() }})" content="{{ parse_less_color(property_variation('metaThemeColor', 'default')) }}" />
            </xf:if>
            <xf:if is="property_variation('metaThemeColor', $xf.style.getAlternateStyleTypeVariation())">
                <meta name="theme-color" media="(prefers-color-scheme: {{ $xf.style.getAlternateStyleType() }})" content="{{ parse_less_color(property_variation('metaThemeColor', $xf.style.getAlternateStyleTypeVariation())) }}" />
            </xf:if>
        <xf:else />
            <xf:if is="property('metaThemeColor')">
                <meta name="theme-color" content="{{ parse_less_color(property('metaThemeColor')) }}" />
            </xf:if>
        </xf:if>
    </xf:if>
<xf:else />
    <xf:if is="property('metaThemeColor')">
        <meta name="theme-color" content="{{ parse_less_color(property('metaThemeColor')) }}" />
    </xf:if>
</xf:if>
Smth. like this would at least allow to have directly empty style property metaThemeColor to skip outputting the meta tag.
 
Back
Top Bottom