Skip to content

Loading…

[Firefox] Forcing fonts in browser eliminates rendering of main toggle in toolbar #629

Closed
ghost opened this Issue · 11 comments

3 participants

@ghost

To recreate: Go to Preferences>Content>Advanced>Allow pages to choose their own fonts>Uncheck>Press Ok

Firefox 35.01; Linux Mint Mate 17.1 64 bit

@chrisaljoudi

Not a bug — the icon is from Font Awesome, and you're disabling the ability to use that.

The alternative would be to use an image, arguably worse at some level.

@chrisaljoudi

@gorhill what do you think? Doesn't seem like it's worth it to move to a PNG/SVG.

@gorhill

I wish all icons were standalone SVGs as this would remove the need to ship with the whole Fontawesome, but I am not an artist. If one would step forward to contribute SVG-based icons that would be great.

@chrisaljoudi

@gorhill I'd be happy to. I'll have to browse through to make sure I catch all the places where FontAwesome is used.

Will have something by the end of today.

@chrisaljoudi chrisaljoudi added the fixing label
@ghost

but I am not an artist

@gorhill Efficient code can be construed as artistic

Will have something by the end of today.

@chrisaljoudi

Right on. Was hoping for something with a dinosaur theme. Maybe a volcano as well. But seriously, thanks guys for looking into it.

@chrisaljoudi chrisaljoudi removed the not a bug label
@ferongr

How about using the equivalent SVG files from https://github.com/encharm/Font-Awesome-SVG-PNG instead?

@gorhill

@chrisaljoudi I would like any changes to go in a separate branch, as I would like to decide whether I agree with the changes before committing them to master.

@chrisaljoudi

@gorhill sure. For just this modification, or just everything from this point on?

@gorhill

Just this, I just want to be sure I like it -- sorry f that sounds rude. I liked the Fontawesome ones (except for the fact this causes dependencies on Fontawesome), except for the big power button, which I thought was too confusing ("power on/off"). I tried to craft an alternative once but I didn't like it and ended up scrapping it. Aside the confusing look, I thought it didn't fill the space well, because circle.

@ghost ghost changed the title from (Firefox) Forcing fonts in browser eliminates rendering of main toggle in toolbar to [Firefox] Forcing fonts in browser eliminates rendering of main toggle in toolbar
@chrisaljoudi chrisaljoudi added browser bug and removed fixing labels
@chrisaljoudi
Owner

Okay.

Disabling fonts on Firefox breaks everything that uses font icons. This is an active bug on Mozilla's tracker and has been 'Assigned'. This is why I'm marking the issue as a browser bug (I personally don't think it's a bug, just an unfortunate limitation, but whatever the Mozilla community thinks is fine).

Individual image files are somewhat of a pain to manage, and are certainly way more of a pain to embed and use.

As far as having to load all of FontAwesome goes, I've been working on a solution that bundles exactly just the icons that are needed. I'll push this soon if there aren't any issues. Meanwhile, there certainly haven't been any efficiency issues brought up about the icons

@gorhill

By the way, I recant my previous "except for the big power button": I like it now, I think the change in color made the difference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.