• Hi there,
    I’m trying this theme out for the first time and like its simplicity and functionality. In particular, it’s great to see a freemium theme where the free version is usable out of the box. Congratulations on a really nice theme.

    I’m making some CSS mods and have spotted a few tiny bugettes on the use of fonts.

    1. Incorrect CSS asking for font that’s not loaded (default settings)
    Inside the CSS for the bbpress fonts used by the theme (in style.css:5260), you have:

    #bbpress-forums fieldset.bbp-form legend {
        ...
        font-family: 'Montserrat', Helvetica, sans-serif;
        font-weight: 500;
        ...
    }

    Unfortunately, only Montserrat with weight 700 has been loaded (in functions.php:189), with:

    if ( 'off' !== $monserrat ) {
                $font_families[] = 'Montserrat:700';
            }

    Indeed, Montserrat would appear only to be available (from Google, at least) in weights 400 and 700: https://www.google.com/fonts#UsePlace:use/Collection:Montserrat

    Solution options:
    – My favoured fix would be to add the 400 weight. That is (in functions.php:189), with:

    if ( 'off' !== $monserrat ) {
                $font_families[] = 'Montserrat:400,700';
            }

    – Alternatively, you could change the CSS to make it all 700, but that’s a little heavy in the context.

    2. Incorrect CSS asking for font that’s not loaded (with “Use safe font” checked)
    If the “Use safe font” option is checked, neither Montserrat nor Lato appear to be loaded. However, the CSS is still asking for them. For example:

    body {
    	font-family: 'Lato', Helvetica, sans-serif !important;
    	...
    }

    The next fallback font is Helvetica, which is a shame, because Open Sans has been loaded by the theme and is more interesting than (the lovely, but ubiquitous) Helvetica.

    Could I suggest that you add Open Sans as a fallback, before Helvetica?

    That is, the CSS above would become:

    body {
    	font-family: 'Lato','Open Sans', Helvetica, sans-serif !important;
    	...
    }

    (with corresponding changes for all the Montserrat declarations too).

    Otherwise, it seems a bit odd going to the bother of loading all the Open Sans fonts in the first place, if we’re not going to use them (and an alternative solution–though not at all my preferred one–would be not to load Open Sans at all).

    3. Typos
    There are some typos in the theme, where Montserrat is written incorrectly as Monserrat.

    Hope this helps to further improve an already great theme!

Viewing 1 replies (of 1 total)
  • Madalin

    (@madalin_themeisle)

    Hey there,

    Thanks for reporting us this issue, I appreciate that. I’ve forwarded your requests to our developers.

    Please let me know if there is anything else I can help out with.

    Best regards,
    Madalin

Viewing 1 replies (of 1 total)
  • The topic ‘A few bugettes in use of fonts’ is closed to new replies.