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

Issues with sites using Emoji Favicon Toolkit #49

Closed
Tracked by #56
eligrey opened this issue Mar 21, 2019 · 4 comments
Closed
Tracked by #56

Issues with sites using Emoji Favicon Toolkit #49

eligrey opened this issue Mar 21, 2019 · 4 comments
Labels
🐌 Bug Something isn't working

Comments

@eligrey
Copy link
Contributor

eligrey commented Mar 21, 2019

Favioli resets the favicon to the default ServiceWorker-cached /favicon.ico on sites that use Emoji Favicon Toolkit, such as https://eligrey.com.

I use EFT here: https://eligrey.com/blog/wp-content/themes/eligrey.com/js/favicon.js

Expected results: Favicon is set to one of: 👨‍🔬 👨‍💻 👨‍🚀 🤯 🤖 ⚡ 🌧️ 🚀 💻 🧠 👻 📜 ☄️ 🛸 🌌 🌠 🛰️ ✨ 🔥 ✈️ ☕️ 🍕 🐱‍💻 and changes to 💬 whenever any input elements get modified.

Observed results: Favicon is set to 🛠️ (default favicon) and never changes.

@bpevs
Copy link
Owner

bpevs commented May 26, 2022

Sorry for slowest turnaround ever. This should be fixed when I release Favioli 2.00. Will close this issue at that time.

Fix is basically to actually check for existing favicon links instead of relying on browser to override favicon in preferred order: d3de7f1

The actual code for ^ will probably change before release, since I want to make sure I still add a favicon for sites with bad urls / 404s. But I will keep this usecase in mind when I edit.

Note: this will effect autofill specifically (the feature that fills in missing favicon). If a user is specifically trying to override the favicon of a website using Emoji Favicon Toolkit, Favioli will side on the user preference.

Video of fix:

49.-.fix.mov

@bpevs bpevs added the 🐌 Bug Something isn't working label May 26, 2022
@Gitoffthelawn
Copy link
Contributor

When v2.00 is released, will it support holographic emoji? I hear they will be the new cool thing in about a decade, so the timing should be about right. 😆 🤣 🤣 🤣 😸

I'm just teasin' ya... take your time... this extension is like a fine wine, except I can afford this extension. 🍷 🤣

@bpevs
Copy link
Owner

bpevs commented May 28, 2022

🗿

Haha the difference is that now I've actually stopped procrastinating on it 😛. Almost feature even with 1.00 now... Only hard thing left is re-implementing the favicon selector ui, since I want to have the support for custom images right off the bat.

@Gitoffthelawn
Copy link
Contributor

Haha the difference is that now I've actually stopped procrastinating on it stuck_out_tongue. Almost feature even with 1.00 now... Only hard thing left is re-implementing the favicon selector ui, since I want to have the support for custom images right off the bat.

Sounds real good. Take your time. Have fun with it.

@bpevs bpevs mentioned this issue Jun 28, 2022
2 tasks
@bpevs bpevs closed this as completed in d3de7f1 Jul 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐌 Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants