CSAILVision / CSAILVision/LabelMeAnnotationTool
Hard to extend due to html_str concatenation
- Dominant language
- JavaScript
- Stars
- 1.5k
- Forks
- 443
- PR merge metrics
- No merged PRs in 30d
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.
Contributor guide
No contributing guide indexed for this repository
Research direction
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.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100