Okay
  Public Ticket #2419401
Font Awesome font-display: swap
Closed

Comments

  •  6
    ctekk started the conversation

    Hi,

    A improvement suggestion:

    add "font-display: swap" to font-awesome and fonts included:

    https://web.dev/font-display/?utm_source=lighthouse&utm_medium=unknown

  •  8,851
    Tahir replied

    Hey Again,

    Thanks for the suggestions .

    Combining the ticket so it's easier for the developer to look into .


    - Lazy load for testimonial (-slider) images.

    Is this something worth getting on the roadmap?

    - Only load used icons of an icon font (currently it loads the complete iconsmind font which is 0.4MB)

    - A function to increase website performance selectable in theme options:

    "Only load VC elements that are used per page/post"

    I saw that in themes like Impreza and it improves loading times.

    Best .



    ThemeNectar Support Team 

  •  6
    ctekk replied

    Hey Tahir,


    Thank you! I think salient has some nice USPs currently, like the most well designed backend builder of all themes.

    If it gets some improvements overall and consistency in VC elements (some vc elements have better options than others, e.g. post grid has lazy load, portfolio has not) I think it can be the best theme in it's class.

    Here are some more improvements:

    • Redo the typograhpy theme settings (or explain it better) - having to select the same font for each item is tedious (or do I only need to set it on one item and leave everything else blank?).
      Also the rem approach from Impreza is really convenient.
      Currently it takes me some time to set up typography for each website. It could be easier, with fewer options (Impreza has like 6 settings).
    • Make it more conventient to implement a custom font. I think step 2 of your guide could be made more convenient in the typography options: https://themenectar.ticksy.com/article/9773/
    • When I implement a custom font I'd like to have more weight options in the typograpy settings drop down (currently limited to bold, bold italic, normal, normal italic).
    • Column in rows can currently be vertically aligned flex-start, middle, flex-end for all columns in a row. It would be convenient if this could be set for each column. E.g. in a 1/2 + 1/2 row I would like the first column to be middle aligned while the last column to be bottom aligned.
    • Multiple VC elements use headings for the title of their items. E.g. Item List uses a heading for the item titles. For SEO purposes it should be possible to <span> or <div> or <p> these item titles. Uncode theme has the most flexibility here. Salient got the VC template overwrite possibility now in 12.0 which is okish. However it's not possible for all VC elements (like for Post Grid I need to edit the loop-markup.php which is not update save)
    • Performance improvements (see attached screenshot what Impreza offers)
    • Native SVG upload support - this is a minor point.


  •  2
    Lars replied

    What is the status of these suggestions by ctekk? Think they can be very beneficial to increase performance.

    In particular "Only load used icons of an icon font" and "Only load VC elements that are used per page/post"

  • Alex Chadwick replied

    Hi Tahir, 

    Have we found a solution for the font-display: swap; as I'm trying to improve me website speed.

    Many Thanks

    Alex