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

모바일 환경에서도 에디터를 사용할 수 있는 기반 PR #1495

Closed
wants to merge 4 commits into from
Closed

모바일 환경에서도 에디터를 사용할 수 있는 기반 PR #1495

wants to merge 4 commits into from

Conversation

misol
Copy link
Contributor

@misol misol commented May 25, 2015

현재는 모바일에서 작성시 nl2br 처리를 피해갈 수 없습니다. 변수(use_editor)를 추가하여 HTML 작성을 가능하게 합니다.
use_editor 값을 Y로 전송할 경우, PC 환경에서 에디터를 사용한 것과 동일하게 해석합니다.

misol added 3 commits May 24, 2015 13:31
현재는 모바일에서 작성시 nl2br 처리를 피해갈 수 없습니다. 변수(use_editor)를 추가하여 HTML 작성을 가능하게
합니다.
@misol
Copy link
Contributor Author

misol commented Jul 8, 2015

@ngleader @bnu 지금은 모바일에서 nl2br이 document나 comment 모듈 단에서 적용되기 때문에 에디터를 사용하더라도 내용이 변형됩니다.
이 PR이 반영되어야 에디터 사용의 의미가 있습니다.

@misol
Copy link
Contributor Author

misol commented Jul 20, 2015

html 소스를 content 항목으로 전달할 때, \r\n 또는 \n 과 같은 개행문자를 모두 제거하면 사실 이 PR을 적용하지 않아도 무리는 없습니다. 하지만, HTML을 개행문자를 포함해서 작성하는 순간 개행문자는 nl2br에 걸리는 상태기 때문에 이 PR이 필요합니다.

이 PR을 아파치, PHP/5.5.15 기반, 그리고 nginx, PHP/5.5.9 기반에서 테스트 했을 때 서버 스크립트 문제가 발생하지 않았습니다
Windows7 Firefox 39.0, 안드로이드 크롬 최신버전, 아이폰5 iOS8 최신버전 사파리에서 테스트 했을때 클라이언트단 오류가 발생하지 않았습니다.

@misol misol closed this Jan 8, 2016
@misol misol deleted the ready-for-mobile-editor-use branch January 8, 2016 12:41
@Lastorder-DC
Copy link

@bnu 모바일 에디터 사용에 치명적인 부분인데 반영이 안될 이유가 있을까요?

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