Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Font Icons instead of embedded Images? #6

Closed
Concept211 opened this issue Nov 29, 2016 · 9 comments
Closed

Font Icons instead of embedded Images? #6

Concept211 opened this issue Nov 29, 2016 · 9 comments

Comments

@Concept211
Copy link
Contributor

I would love to implement lightweight font icons to reduce the file size a bit more of the script (to hopefully be able to run it on GoDaddy) and also display sharp, vector-based icons. I know it's not possible to embed the actual font icons into a single script, but you are able to simply load an external CDN like the Font Awesome library. I'm not a huge fan of FA, but it would provide a quick and easy external solution without having to embed anything. Plus, they're about to release v5 of their collection, so there should be plenty of new icons to work with. I know a lot of changes would be necessary due to all of the icons and buttons currently being made up of IMG elements, but I think it would benefit the script.

What do you think?

@misterunknown
Copy link
Owner

Well, the huge file size of the script is indeed a problem. Unfortunately, loading an external font from a CDN is not an option for me, because sometimes I use the IFM in environments where I'm not connected to the internet, or sit behind a restrictive proxy.

At the moment I work on a new version. To lower the filesize I will strip out some infantile features like different editor themes and try to reduce the dependencies like JQueryUI. Maybe I will also replace the ace editor by a more lightweight one. In addition to that I will look for some nice SVG icons, which can replace the base64 encoded pngs.

Anyway, thanks for your input!

@Concept211
Copy link
Contributor Author

Sounds great! Thanks for your hard work. If there's anything I can do to contribute, please let me know.

@misterunknown
Copy link
Owner

Little update:

I found a way to directly embed a font into the single ifm file. At the moment I design a new frontend using bootstrap and some nice icons from fontawesome. While diggin around searching for a sufficient icon set I found this awesome site, which allowes one to create a custom set of icons. I think the update is ready by the end of the week.

@Concept211
Copy link
Contributor Author

Great news, thanks!

@Concept211
Copy link
Contributor Author

Side note:
Remember I was having issues with GoDaddy where I couldn't upload the file because it was too large? Well, it turned out to be an issue with the old server that my account was running on. They just recently set me up on a newer Linux server due to some other issues that I was encountering and I just tested IFM today and it works perfectly! ;)

@misterunknown
Copy link
Owner

Thanks for the update! Nice to hear that it runs with GoDaddy servers.

I just pushed my new version with bootstrap and font icons. Take a look! :)

@Concept211
Copy link
Contributor Author

It looks great and at half the size! Great job!

One thing I'm really missing though is the image preview on hover. I'm sure it's on your list but I just wanted to bring it up.

@misterunknown
Copy link
Owner

Thanks. I'm glad you like it. Only the font icons could be more sane. But I didn't find any better icons, yet.

The image preview, as well as the multiselect function, will be added soon :)

@misterunknown
Copy link
Owner

Image preview and multiselect will be added in the next release, so I close this issue for now ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants