kijin / xe-zipperupper

XE ZipperUpper! 애드온
GNU Lesser General Public License v2.1
1 stars 1 forks source link

ie10 이하에서만 작동하는 옵션이 있으면 어떨까요? #12

Closed ajkj closed 9 years ago

ajkj commented 9 years ago

문제가 발생하는 ie10 이하에서만 작동하는 옵션이 있으면 어떨까 해서 PR을 올려봅니다.

보통은 페이지별로(메인, 글목록, 글쓰기, 글읽기 등등...) CSS/JS가 조금씩 차이가 나는데요, 이 경우 zipperupper를 일괄적으로 적용해 버릴 경우, 페이지를 전환할때 마다 모든 CSS/JS를 새로 다운받아야 하는게 조금 아쉬운것 같아요.

그래서 CSS 갯수 문제가 발생하는 IE10 미만에서만 작동하도록 하는 옵션을 추가해 보았습니다.다. 이렇게 할 경우 모던 브라우저에서는 그때 그때 필요한것만 다운로드 받을 수 있어서 효과적이지 않을까요?

kijin commented 9 years ago

혹시 ie11도 호환성보기 모드로 사용하면 CSS 갯수 제한이 발생하나요? ie11은 항상 괜찮나요?

ie10도 원래 괜찮아야 하는데 가끔 문제가 발생한다는 것을 보면, 호환성보기의 영향을 받는 게 아닌가 싶은데...

ajkj commented 9 years ago

XE는 기본적으로 IE 호환성 보기 모드를 강제적으로 꺼주기에 호환성보기 관련이 문제라면 발생하지 않아야 정상일것 같가요. https://github.com/xpressengine/xe-core/blob/master/common/tpl/common_layout.html#L14

kijin commented 9 years ago

별짓을 다해도 계속 켜져있는 경우도 있더군요 ㅠㅠ

아무튼 IE11부터는 user-agent에 MSIE라는 문자열이 들어 있지 않다는 점에 착안해서, MSIE라는 문자열이 들어 있으면 무조건 IE10 이하로 취급하도록 했습니다.

IE11이라도 호환성보기 모드 사용시 MSIE라는 문자열이 다시 추가되므로 애드온이 정상 작동할 거예요.