Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author Michael Bourne

    (@michaelbourne)

    Hey @alriksson

    Font-display has no valid values for icon fonts. The default behaviour of ‘block’ will apply.

    Thread Starter Rookie

    (@alriksson)

    hmm, I know what you mean but I do not agree.

    Thread Starter Rookie

    (@alriksson)

    Also, wonder why the same files are x2 loaded but with different paths and hash numbers? Is it necessary to load both can we not combine them?

    Thread Starter Rookie

    (@alriksson)

    You can add and would like to see

    font-display: swap;

    to the generated css file

    • This reply was modified 5 years, 9 months ago by Rookie.
    Plugin Author Michael Bourne

    (@michaelbourne)

    Hey @alriksson

    There’s no point in adding swap, as icon font’s have nothing to actually swap to like text fonts which would swap to system fonts during load.

    Files are loaded twice when logged in as an admin, because I wasn’t able to get the Elementor editor action working for loading a CSS file when I first released the plugin. It shouldn’t happen for guests.

    Thread Starter Rookie

    (@alriksson)

    There’s no point in adding swap, as icon font’s have nothing to actually swap to like text fonts which would swap to system fonts during load.

    Still, improving the render time of 200-300ms per package which is why I think 1 line of code is worth doing. Added it myself in the precompiled styles.

    Files are loaded twice when logged in as an admin, because I wasn’t able to get the Elementor editor action working for loading a CSS file when I first released the plugin. It shouldn’t happen for guests

    You′re right found out this later on as well that it is only upon logged in users. Thanks.

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘add support for font-display’ is closed to new replies.