CSAILVision / CSAILVision/LabelMeAnnotationTool
Hard to extend due to html_str concatenation
- Langage dominant
- JavaScript
- Étoiles
- 1.5k
- Forks
- 443
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
The tool's javascript is written in an very old style. It uses string concatenation to form DOMStrings, which makes it not extensible at all.
For example, special characters can't be handled unless completely rewriting of all the related code.
I tried to do that and, unfortunately, something broke there. I found the codes coupling with each other heavily, making it hard to troubleshoot.
BTW, the repository seems not under maintenance any more (am I wrong?). I wonder if someone's going to refine it for better extensibility.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Start by locating the JavaScript that concatenates strings into DOMStrings and map the related code paths that make special characters difficult to handle. Review the coupling before deciding on a refactoring scope; done should mean the affected DOM construction is more extensible and special-character handling no longer requires rewriting all related code.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- frontend
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 20/100