CSAILVision / CSAILVision/LabelMeAnnotationTool

Hard to extend due to html_str concatenation

Open
#68 0 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.