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

모바일 레이아웃에서도 meta tag를 수정할 수 있도록 합니다. #1475

Closed
wants to merge 2 commits into from
Closed

Conversation

misol
Copy link
Contributor

@misol misol commented May 15, 2015

mobile_layout.html에는 common_layout과 달리 메타태그를 Context::addMetaTag 로 추가할 수 없습니다. 모바일에서도 이를 가능하게 하는 PR 입니다.
모바일에서도 SEO등 meta tag를 사용할 일은 많습니다.
반영 되길 기원합니다!!

@ghost ghost added the type/enhancement label May 18, 2015
@ghost ghost added this to the next-2015-24 milestone May 18, 2015
@ghost ghost self-assigned this May 18, 2015
@ghost ghost modified the milestones: next-2015-24, next-2015-28 Jun 9, 2015
@ghost ghost modified the milestones: next-2015-28, next-2015-32 Jul 6, 2015
@misol
Copy link
Contributor Author

misol commented Jul 20, 2015

이 PR을 아파치, PHP/5.5.15 기반, 그리고 nginx, PHP/5.5.9 기반에서 테스트 했을 때 서버 스크립트 문제가 발생하지 않았습니다.
클라우드 플레어를 이용했을 때 문제가 발생하지 않았습니다.
Window7 Firefox 39.0 에서 모바일 페이지를 테스트 했을때 클라이언트단 오류가 발생하지 않았습니다.
이 PR을 적용하고 안드로이드 폰과 아이폰에서 테스트 했을때 문제가 발생하지 않았습니다.
안드로이드 버전 5.0.1, 갤럭시S5, Chrome 브라우저 최신버전(2015년 7월 20일 기준)에서 테스트 했습니다.
아이폰은 아이폰5 iOS8 최신버전(2015년 7월 20일 기준)에서 테스트 했습니다.
테스트 도중 구성한 페이지들에서 문제가 발생하지 않았습니다. 구성한 페이지는 게시판 목록, 게시판의 본문글, 위젯 페이지, 외부 페이지 등이 포함 되었습니다.

@ghost ghost modified the milestones: next-2015-32, next-2015-37 Aug 18, 2015
@ghost ghost modified the milestones: next-2015-37, next-2015-41 Sep 21, 2015
@ghost ghost modified the milestones: next-2015-41, next-2015-46 Oct 16, 2015
@misol misol closed this Jan 8, 2016
@misol misol deleted the mobile-meta-control branch January 8, 2016 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant