fix: loadIcon always console warn

[geforseN]

🔗 Linked issue

resolves #271

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This PR addresses an issue where the console warning [Icon] loading icon <name> timed out after <timeout>ms would be shown, even though the icons were loaded successfully. This happened due to incorrect handling of icon loading timeouts.

Changes introduced:

1. Improved error handling: Now, if _loadIcon fails to load the icon, a specific warning message [Icon] failed to load icon <name> will be logged, giving more visibility into failed loads.
2. Timeout handling fix: The timeout warning [Icon] loading icon <name> timed out after <timeout>ms will now only appear if the icon fails to load within the specified timeout period.
3. Cleanup with finally: Ensures that the timeout warning is cleared properly whether the icon is loaded successfully or fails.

📝 Additional context

If you only want the bug fix (c072229e163fcf738c22fe1bfea9178434651419), you can pick only that specific commit

[JessicaSachs]

Heh, you beat me to it. Was about to submit a patch. 🙈