Needlworks / Textcube

Textcube : Brand yourself! / Personalized web publishing platform with multi-user support

http://www.textcube.org

Other

208 stars 55 forks source link

modal pop-up dialog for comment reply, delete, modify UI #1725

Open inureyes opened 9 years ago

inureyes commented 9 years ago

Summary

modal pop-up dialog for comment reply, delete, modify UI
Using jQuery

inureyes commented 9 years ago

Requirements

~~Proper jQuery plugin (should handle iframe)~~ -> bPopup
~~Dialog element in DOM (#tcDialog?)~~ -> #tcDialog
openWindow function -> openDialog function (in common2.js)
Error handling
Close modal pop-up at the end of tasks (e.g. comment to comment saved)

zvuc commented 9 years ago

페이지 내에 팝업을 띄우게 된다면 사용자가 디자인을 쉽게 변경할 수 있도록 CSS로 선택 가능한 클래스명을 지정하거나 skin.html에 해당 부분을 넣어 스킨의 디자인에 따라 default레이아웃 구조를 변경할 수 있도록 할 수 있게 함이 어떨까 합니다.

achimnol commented 9 years ago

드디어 해묵은(...) 논의거리였던 CSS namespace의 필요성이 나오는군요. "textcube-"와 같은 prefix를 붙여서 텍큐 쪽이 제공하는 기본 UI 디자인을 해두고 사용자가 원하면 override할 수 있게 하자는 아이디어....ㅎㅎ 그리고 가능하면 id보다는 class를 위주로 사용했으면 좋겠습니다. 다이얼로그 창 안에서 또 다이얼로그가 뜬다거나 이런 상황도 생길 수 있으니까요.

zvuc commented 9 years ago

네, 애초에 새로운 규격을 추가해버리면 그 부분이 없었던 기존 스킨은 못 쓰게 되어버리니 기본 스타일을 출력하고 CSS에서 잡아서 오버라이드만 할수 있어도 충분하지 않나 싶습니다 ㅋㅋ iframe만 아니라면야(..)

inureyes commented 9 years ago

@achimnol @zvuc 현재 구현은, 팝업을 호환성을 위해 그대로 남겨두고 iframe을 사용해서 modal dialog를 얹어놓은 구현 형태입니다. 따라서 기존 디자인을 그대로 따라가게 되어 있지요. ;;

2.0에서 CSS namespace 문제를 해결합시다. 1.10에 뭔가 더 정성을 쏟으면 벌써 오 년 넘게 이어지고 있는 '두 개발트리' 상태에서 못 벗어날 듯... ㅎㅎ

inureyes commented 9 years ago

When using mobile-support skins (e.g. periwinkle), pop-up does not call.

With mobile-support skin, textcube recognize current display mode as desktop. Therefore comment-to-comment script call modal pop-up.

inureyes commented 9 years ago

Disable ESC / modal click to prevent blow-up comments by unintended clicks.