Needlworks / Textcube

Textcube : Brand yourself! / Personalized web publishing platform with multi-user support
http://www.textcube.org
Other
208 stars 55 forks source link

Bring apply button back to TTML image property editor #1743

Open zvuc opened 9 years ago

zvuc commented 9 years ago

screen shot 2015-01-12 at 11 04 50 am

본래 있었다가 언젠가부터 사라진걸로 기억하는데요, 내장 이미지 업로더로 삽입한 이미지를 선택했을때 뜨는 모달박스 에 '적용' '취소' 버튼이 사라져서, 자막이나 대체텍스트등을 입력한 후 적용을 하려면 키보드로 엔터를 눌러줘야 반영이 된다는 문제가 있습니다. 본래 자동으로 저장이 되는 기능이 들어가서 버튼이 없어진것 같은데 이게 됐다가 안됐다가 하는것같네요. 입력칸에 텍스트를 입력한 뒤에 엔터를 누르지 않고 이미지 선택을 해제하면 그냥 빈칸으로 리셋되어버립니다.

간단한 문제같은데 한번 확인해보심이 좋을것 같습니다

inureyes commented 9 years ago

아마 CSS 문제가 아닐까 짐작을..

zvuc commented 9 years ago

저도 처음에 그런줄 알고 확인해봤는데 html 소스에 마크업 자체가 없더라구요(..)

inureyes commented 9 years ago

@zvuc 마크업은 javascript에서 만들어져서 들어가는 부분인데, 확인해 보겠습니다!

inureyes commented 9 years ago

버튼은 아예 없는게 맞네요. 포커스가 떠났을 때 내용 저장 부분을 살펴 보겠습니다.

inureyes commented 9 years ago

지금 테스트 해 보니 포커스 떠날 때 엔터를 입력하지 않아도 내용이 자동 갱신 되는데요, 혹시 자동 갱신이 되지 않는 시나리오가 있으시면 테스트해 볼 수 있도록 알려주세요. 전 MacOSX 10.10.1 / Safari 에서 우선 테스트했습니다. :dancer:

inureyes commented 9 years ago

onkeyup 이벤트가 입력박스에 붙어 있고, setProperty 를 실행하는 구조입니다. 정상적인 경우라면 setProperty가 키를 뗄 때 마다 불리면서 속성을 실시간으로 변경하게 됩니다. 버튼을 붙이는거야 버튼 붙이고 setProperty를 부르도록 만들면 되긴 합니다만, 현재 왜 자동 저장이 정상적으로 돌아가지 않는지를 먼저 파 보겠습니다.