eclipse-theia/theia

Use "Broken Icon Reference" when not finding icons from contributions

Open

#8 240 ouverte le 27 juil. 2020

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)TypeScript (18 676 stars) (2 478 forks)batch import
contributor experienceenhancementgood first issuehelp wanted

Description

Feature Description:

We have cases like this (https://github.com/eclipse-theia/theia/issues/7634) where we can't resolve an icon that is referenced in a contribution. It would be helpful to return an icon that signifies "Icon not found" and display that in the place of the desired icon. Currently, there is no indication in the UI that an icon could not be loaded and where it would be.

Guide contributeur

Stack technique
typescriptreact
Domaine
frontenddeveloper experience
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Familiarity with Theia's icon contributions
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Investigate icon resolution in Theia's frontend, specifically the icon service (likely in packages/core/src/browser/icon service.ts). Look for where icon contributions are resolved and displayed. Consider adding a fallback icon (e.g., a 'broken' icon class) when a referenced icon cannot be found. Review related issue #7634 for context.