dyerc / jQuery-Font-Chooser

A custom combo box to allow font selection from a preset list, whilst previewing the font
MIT License
30 stars 16 forks source link

Invalid font style declaration #10

Open vishy93 opened 7 years ago

vishy93 commented 7 years ago

@CD1212 Using this plugin I have noticed the following issues: 1) Using and selecting a font such as the Goudy Bookletter 1911https://fonts.google.com/specimen/Goudy+Bookletter+1911?selection.family=Goudy+Bookletter+1911 there is an issue where the option in the chooser becomes blank and no style is applied for it. See the error in action here: https://jsfiddle.net/vish93/yuqdj04f/1/

**See picture for the font chooser ul element showing the missing font family:**
               [https://postimg.org/image/52g7ipqrp/](url)

        **Note:- The blank looks to be an issue during the selection process in the plugin, Initially starting 
                    with said font displays the text correctly. Mind just not in the font itself.** 

2) Fonts inline are not being inlined correctly and repetitive declarations: For illustration: <li style="font-family: "Arial Black", "Arial Black", Gadget, sans-serif;">Arial Black</li>

Inner quotes should be single-quotes.