Cloud-Platform icon indicating copy to clipboard operation
Cloud-Platform copied to clipboard

Live Editor is not using latest FontAwesome

Open cocarrig opened this issue 2 years ago • 1 comments

Describe the bug When editing custom screens in the live editor, the Fonts Awesome Fonts being used are on v6 and are outdated. This results in some deviation of display between the end device display.

To Reproduce Steps to reproduce the behavior:

  1. Create custom screen.
  2. Add block to custom screen that will use icons, this can be a calendar with icons entered, or a homeassistant block/ fetch block with icons added.
  3. Note that icons rendered in the live editor layout have differences in style with the the preview function, and end device used.

Expected behavior Live editor should be using latest Fonts Awesome Fonts available.

Screenshots N/A

Desktop (please complete the following information):

  • N/A

Smartphone (please complete the following information):

  • N/A

Additional context Identified in client case 38578.

cocarrig avatar Apr 01 '22 15:04 cocarrig

We will need to take some further consideration for this. The new version of FontAwesome tracks each page-load, and we're limited to 1M/month. If we update the screens to use this, we may very well go over 1M. At the very least we will need to monitor usage. If/when we go over the limit, we can self-host the fontawesome files on static.dakboard.com.

danking6 avatar Apr 06 '22 13:04 danking6

All associated icon selectors and live editor will be moved to FA 5.15 to be made consistent with the actual DAKboard screens.

cocarrig avatar Mar 09 '23 15:03 cocarrig

  • v6 Kit loads are counted as page loads, not individual icon usages
  • v6 Kit loads in the code appear only in options and screen-layout-editor layouts
  • v6 font family is only set on .checkbox styled items

Given these, we have turned off all kit loads, reinstated the v5 links and reverted Font Awesome 6 Pro references to 5.

Dan-Peck avatar Mar 13 '23 13:03 Dan-Peck

Merged in @ 09ac20a and added to the queue for upcoming release to the live site.

Dan-Peck avatar Mar 23 '23 19:03 Dan-Peck