CSAILVision / CSAILVision/LabelMeAnnotationTool

Hard to extend due to html_str concatenation

未关闭
#68 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
JavaScript
星标
1.5k
派生
443
PR 合并指标
30 天内没有已合并 PR

描述

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.

贡献指南

这个仓库没有索引到贡献指南

调研方向

首先定位将字符串拼接成 DOMStrings 的 JavaScript,并梳理使特殊字符难以处理的相关代码路径。在决定重构范围之前,先审查代码耦合;当受影响的 DOM 构造更具可扩展性,且特殊字符处理不再需要重写所有相关代码时,即可视为完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
javascript
领域
frontend
Issue 类型
重构
难度
5/5
预计耗时
一周以上
活跃度
停滞
描述清晰度
需要澄清
新手友好度
20/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。