Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Non-Flash HTML5 based file uploader for editor #1860

Closed
wants to merge 12 commits into from

Conversation

zvuc
Copy link
Contributor

@zvuc zvuc commented Apr 2, 2017

3년전에 건의한 html5기반 Non-flash 멀티 파일업로더(#1730)에 관하여, 도저히 현실적으로 근 시일 내에 개선이 이루어질것 같지 않아보여서 직접 어떻게 할수 있지 않을까 하여 뜯어고치다가 일단 작동하는 멀티업로더 구현을 했습니다.

아주 핵심적인 필요만 충족하는 구현인 관계로 코드 퀄리티나 자잘한 문제가 있으리라 생각됩니다만 일단 나름 테스트를 해본 바로는 최신 버전의 크롬, 파이어폭스, 사파리에서는 제대로 사용 및 동작이 가능한것으로 확인했습니다. 간단한 프로그레스 바도 넣었습니다만 여러개의 파일 동시 업로드시 각 파일별 프로그레스가 순차적으로 보이게 해야하는데 (5개 중 3번째 업로드중...) 이건 끝내 구현을 못 했습니다 (제 자바스크립트 실력의 한계인듯 합니다 ㅠㅠ)

어찌됐든 현재 플래시가 보안성 측면에서나 사용성 면에서 어떻게든 사용하지 않는 것이 바람직 한것이라 생각되어, 과감하게 기존 플래시 업로더 관련된 설정이나 출력 부분을 완전히 떼고 새 것으로 교체했습니다만, 한번 확인해보시고 괜찮으시면 PR 머지하시거나, 또는 마스터에 가져가시면 될것 같습니다.

사실 계속 이미 개발이 중단된 1.10 기반으로 작업해서 PR날리는게 좀 죄송스럽기도 한데, 마스터 기반으로 작업하려 시도했지만 최신 커밋 기준 제대로 동작하는 2.0 브랜치 작업환경을 설정하는데 실패했기도 하고, 아직 일반 사용 용도로는 2.0 알파가 권장되지 않는 이상 1.0버전 대 최신 브랜치인 1.10 기반으로 작업하게 된 점 양해 부탁드립니다.

추가적으로 bf05d81 커밋에서는 TinyMCE에디터에서 불러들이던 override.css의 부분을 스킨 부분으로 옮겼는데, 업로더 부분에 관여하는 코드가 타 admin 스킨을 이용시 부적절하게 적용이 되기 때문에, 해당 부분을 없애고 기본 제공 어드민 스킨인 canon의 editor부분에서 override하도록 옮겼습니다.

@inureyes
Copy link
Member

inureyes commented Jun 3, 2017

리뷰해 보겠습니다~ 늦어서 죄송합니다. ㅠ

@zvuc zvuc closed this Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants