Skip to content

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

Closed
misol wants to merge 4 commits into
xpressengine:developfrom
misol:ready-for-mobile-editor-use
Closed

모바일 환경에서도 에디터를 사용할 수 있는 기반 PR#1495
misol wants to merge 4 commits into
xpressengine:developfrom
misol:ready-for-mobile-editor-use

Conversation

@misol

@misol misol commented May 25, 2015

Copy link
Copy Markdown
Contributor

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

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

misol commented Jul 8, 2015

Copy link
Copy Markdown
Contributor Author

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

@misol

misol commented Jul 20, 2015

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown

@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.

2 participants