-
Notifications
You must be signed in to change notification settings - Fork 7
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
[SP1] SEO 향상을 위한 meta tag, open graph #188
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
너무너무 훌륭합니다!!! 🚀🚀🚀
src/pages/_app.tsx
Outdated
content="SOPT는 IT와 벤처 창업에 뜻이 있는 대학생들이 모인 국내 최대 규모의 대학생 연합 IT 벤처 창업 동아리입니다." | ||
/> | ||
<meta name="title" content="SOPT" /> | ||
<meta name="keyword" content="IT, 창업, 개발, 대학생, 동아리"></meta> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 친구도 self closed 태그 형식이면 좋을 것 같아요 .!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네! 알려줘서 고마워요~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
와 접근성 점수가 엄청 상승했네요!! 굳입니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
한 번 더 approve 합니다~~ 최고!!!
@@ -2,7 +2,7 @@ import { Head, Html, Main, NextScript } from 'next/document'; | |||
|
|||
export default function Document() { | |||
return ( | |||
<Html> | |||
<Html lang="ko" dir="ltr"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ko 로 설정하신 이유가 있나요??
korean 으로 설정하면 스크린리더가 한국말에 맞춰 코드들을 읽게 돼요.
그러면 alt="hello" 를 "에이치 디 엘 엘 오" 라고 읽을 것 같아요
코드 한 줄 한 줄 적을 때도 블로그 그대로 따라하는 게 아니라 다른 사이드 이펙트가 생길지, 이걸 꼭 해야하는 건지 고민하는 게 중요해요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
솝트 지원자 중에 외국인은 없을 것이라고 생각했어요! 코드에서 alt 태그로 해당 img의 타이틀로 지정해둔 경우가 대부분이라서 한글로 읽히는 것이 더 적절할 것 같았어요.
다시 확인해보니까 영어로 두루뭉실하게 적혀있는 코드들도 있어서 해당 코드들은 리팩토으로 추가로 작업하겠습니다!
리뷰 감사합니다!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
만약 영어와 한글을 혼재하게 사용한다면 lang 속성을 지정하지 않는것이 나을까요?
하나의 언어를 통일적으로 사용하는게 좋을까요?
의견이 궁금합니다...!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어떻게 생각하시나요??? 어떤 장단점이 있을 것 같으세요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저는 하나의 언어로 통일하는게 더 좋다고 생각해요!
하나의 언어로 통일할수록 해당 속성에 대해 자세하게 명명할 수 있어서요.
다른 분들의 의견도 궁금해서 물어봅니다~
Summary
close #180
누락되어있던 meta tag를 추가하고 open graph 문구 및 이미지를 변경하였습니다!
Screenshot
[open graph]
[light house 점수 향상]
Comment