몇가지 수정 건입니다

[mulder21c]

1. npm script 가 정상동작하지 않는 건이 있습니다.

   font 쪽은 이전 PR에서 해결된 건이 있기에 해당 수정을 그대로 유지하고, js 파일들을 uglify 하고 concat 하는 부분에서 세미콜론으로 연결된 명령이 동작하지 않는 부분에 대한 수정

2. atom.xml에 대한 link element의 rel 값 수정

   HTML 5 명세 상, link element의 rel 값에는 alternative가 없고, alternate를 사용하려 했던 것으로 판단하여 해당 값으로 수정해두었습니다.

3. viewport에서 maximum scale 값이 1로 설정된 부분을 제거

   이 부분은 접근성 관련 이슈로, maximum scale이 1로 설정될 경우 저시력 장애자가 볼 경우, 화면을 확대할 수 없는 문제가 발생됩니다. 비단 저시력 장애자 뿐 아니라, 글자가 작은 내용을 볼 경우 해당 설정으로 인해 핀치줌을 사용할 수 없기 때문에 불편함이 발생되기 때문에 해당 설정을 제거하는 방향이 좋을 듯 합니다.

4. canonical URL과 og:url 일치하도록 수정

   canonical URL에 index.html 을 empty string으로 치환하는 코드가 존재하는 반면, opengraph url는 index.html을 포함하는 문제로 Facebook Share Debugger에서 오류가 발생합니다. opengraph url를 canonical URL과 마찬가지로 path까지의 URL로 치환되도록 수정되었습니다.

5. 세로 100% 레이아웃 적용

   포스트가 짧을 경우에는 footer와 sns share 영역 사이에 넓은 간격이 발생되기 때문에, 포스트가 짧을 경우 footer가 바닥에 위치하고 콘텐츠 영역이 길어지도록 수정되었습니다.

6. LiveRe 플랫폼 추가

   국내 소셜 덧글 플랫폼인 LiveRe를 적용할 수 있도록 코드가 추가되었습니다. disqus 보다는 국내용 SNS를 활용할 수 있는 면이 많아서 (카카오, 네이버 등등) 국내용 덧글 플랫폼으로는 가장 적절하지 않을까 싶습니다.

[HyunSeob]

여러가지 실수를 바로 잡아주셨네요!! PR 감사합니다! 👍

[HyunSeob]

3번 같은 경우는 말씀하신 부분이 맞긴한데, 저는 핀치 줌/줌아웃을 해주는 걸 좋아하지 않아서요. 이 부분은 검토 후 다른 방법으로 해결하는 게 좋을 것 같아요. 네이버 뉴스처럼 글씨 크기를 바꿀 수 있는 기능을 제공한다던지..

5번도 좋은데요, Flex를 쓰면 어떨까 합니다. IE9 지원은 그냥 버리구요.

[mulder21c]

@HyunSeob

음.. 3번의 문제는 현재 구조에서는 폰트 사이즈를 증가시키는 방식으로 하자면 연관된 스타일 구조들을 (fixed position 등) 새로 잡아야할 듯 합니다 ^^a PC 브라우저에서도 html의 font-size를 증가시킨 것과 브라우저 자체 zoom 을 증가시킨 것과의 차이가 꽤 많은 부분 이슈가 발생되거든요 ㅠㅠ

해당 부분은 config에서 pinch zoom-in/out을 설정 - 해제 하는 방법은 어떨까요? ^^ 테마 사용자에게 선택권을 주어서 해결하는 것도 나쁘지 않을 것 같습니다.

Jade에서 분기처리만 추가해주면 특별히 손을 보지 않아도 될 것 같구요 ㅎㅎ

5번의 경우 flex는 IE 쪽과 Safari 에 보고된 이슈들이 많아서 적용을 해보지는 않았는데, 몇 가지 알려진 버그나 이슈들이 있기는 하지만 시도해보면 좋겠네요 ^^

당분간은 짬을 내기가 또 쉽지않아서 시간이 좀 걸리겠지만 ㅠ 적용해보고 테스트 이후에 별도로 PR 드리도록 하겠습니다 ㅎ

[HyunSeob]

5번은 당장 flex로 전환하기에 시간이 좀 오래걸릴테니 제가 추후에 작업하는 것으로 할게요. 3번은 제가 스스로 고민 해보고 적용하도록 하겠습니다. PR 정말 감사드리고 수고하셨습니다!! 🙇