The "if" means the default is never used

[davorg]

This is a bug fix.

Summary

Fixes the social share include so the default text works as expected.

Context

The current code looks like this:

  {% if site.data.ui-text[site.locale].share_on_label %}
    <h4 class="page__share-title">{{ site.data.ui-text[site.locale].share_on_label | default: "Share on" }}</h4>
  {% endif %}

The if statement means that the default ("Share on") is never used. And I assume that's what most people would want to see.

Also, without the <h4>, we get a HTML validation warning - as the <section class="page__share"> has no header.

Just removing the if/endif lines fixes these problems without changing the behaviour.