Please consider inlining image resources as base64 #5

Closed
stighackvan opened this Issue Mar 20, 2013 · 6 comments

Comments

Projects
None yet
5 participants
@stighackvan

the chrome extension singlefile has a clever way of inlining all resources for a web page into a single (terribly huge) html file...

among the tricks used, images get inlined as base64 so they don't need to be downloaded or saved in separate files... the incantation looks a bit like this...

background:url('data:text/plain;base64,iVB[and.lots.more.base64-encoded.image.data.goes.here]')

in case of IITC becoming a refugee extension, it would be quite nice if it were more self-contained and didn't depend upon image resources out on the net... and, the file size burden should be small for small icons.

@stellarhopper

This comment has been minimized.

Show comment Hide comment
@stighackvan

This comment has been minimized.

Show comment Hide comment
@stighackvan

stighackvan Mar 20, 2013

Oh beautiful! awesome!!!! looks to be nicely integrated into the python build system.... great eye, stellarhopper!!

Oh beautiful! awesome!!!! looks to be nicely integrated into the python build system.... great eye, stellarhopper!!

@Fragger

This comment has been minimized.

Show comment Hide comment
@Fragger

Fragger Mar 20, 2013

Contributor

I have more for getting the plugins doing the same in my moving-forward branch

Contributor

Fragger commented Mar 20, 2013

I have more for getting the plugins doing the same in my moving-forward branch

@breunigs

This comment has been minimized.

Show comment Hide comment
@breunigs

breunigs Mar 25, 2013

Contributor

probably need to speed this up: https://ingress-verifier.appspot.com/ (essentially the same message they sent me, so I would assume this is compancy policy)

Contributor

breunigs commented Mar 25, 2013

probably need to speed this up: https://ingress-verifier.appspot.com/ (essentially the same message they sent me, so I would assume this is compancy policy)

@Fragger

This comment has been minimized.

Show comment Hide comment
@Fragger

Fragger Mar 25, 2013

Contributor

It is done just need #54 merged. All that really needs to be done after that is #58 and that is not crucial

Contributor

Fragger commented Mar 25, 2013

It is done just need #54 merged. All that really needs to be done after that is #58 and that is not crucial

@jonatkins

This comment has been minimized.

Show comment Hide comment
@jonatkins

jonatkins Mar 25, 2013

Collaborator

All looks good - will push out a new public release tonight

Collaborator

jonatkins commented Mar 25, 2013

All looks good - will push out a new public release tonight

@jonatkins jonatkins closed this Mar 25, 2013

jonatkins pushed a commit that referenced this issue Dec 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment