CSAILVision / CSAILVision/LabelMeAnnotationTool

Hard to extend due to html_str concatenation

Ouverte
#68 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.