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

[SP1] SEO 향상을 위한 meta tag, open graph #188

Merged
merged 9 commits into from
Sep 21, 2023
Merged

Conversation

f0rever0
Copy link
Contributor

Summary

close #180
누락되어있던 meta tag를 추가하고 open graph 문구 및 이미지를 변경하였습니다!

Screenshot

[open graph]
스크린샷 2023-09-18 오후 1 50 18

[light house 점수 향상]
image

Comment

@f0rever0 f0rever0 self-assigned this Sep 19, 2023
@pull-request-size pull-request-size bot added size/M and removed size/S labels Sep 19, 2023
Copy link
Member

@SeojinSeojin SeojinSeojin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

너무너무 훌륭합니다!!! 🚀🚀🚀

content="SOPT는 IT와 벤처 창업에 뜻이 있는 대학생들이 모인 국내 최대 규모의 대학생 연합 IT 벤처 창업 동아리입니다."
/>
<meta name="title" content="SOPT" />
<meta name="keyword" content="IT, 창업, 개발, 대학생, 동아리"></meta>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 친구도 self closed 태그 형식이면 좋을 것 같아요 .!!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네! 알려줘서 고마워요~

Copy link
Member

@solar3070 solar3070 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

와 접근성 점수가 엄청 상승했네요!! 굳입니다!

Copy link
Member

@SeojinSeojin SeojinSeojin left a 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">
Copy link
Member

@joohaem joohaem Sep 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ko 로 설정하신 이유가 있나요??
korean 으로 설정하면 스크린리더가 한국말에 맞춰 코드들을 읽게 돼요.
그러면 alt="hello" 를 "에이치 디 엘 엘 오" 라고 읽을 것 같아요

코드 한 줄 한 줄 적을 때도 블로그 그대로 따라하는 게 아니라 다른 사이드 이펙트가 생길지, 이걸 꼭 해야하는 건지 고민하는 게 중요해요!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

솝트 지원자 중에 외국인은 없을 것이라고 생각했어요! 코드에서 alt 태그로 해당 img의 타이틀로 지정해둔 경우가 대부분이라서 한글로 읽히는 것이 더 적절할 것 같았어요.
다시 확인해보니까 영어로 두루뭉실하게 적혀있는 코드들도 있어서 해당 코드들은 리팩토으로 추가로 작업하겠습니다!

리뷰 감사합니다!!

Copy link
Contributor Author

@f0rever0 f0rever0 Sep 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

만약 영어와 한글을 혼재하게 사용한다면 lang 속성을 지정하지 않는것이 나을까요?
하나의 언어를 통일적으로 사용하는게 좋을까요?

의견이 궁금합니다...!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

어떻게 생각하시나요??? 어떤 장단점이 있을 것 같으세요??

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저는 하나의 언어로 통일하는게 더 좋다고 생각해요!
하나의 언어로 통일할수록 해당 속성에 대해 자세하게 명명할 수 있어서요.

다른 분들의 의견도 궁금해서 물어봅니다~

@pull-request-size pull-request-size bot added size/L and removed size/M labels Sep 20, 2023
@f0rever0 f0rever0 merged commit abca471 into develop Sep 21, 2023
1 check passed
@f0rever0 f0rever0 deleted the feat/#180-meta-tag branch September 21, 2023 14:38
@f0rever0 f0rever0 restored the feat/#180-meta-tag branch September 21, 2023 14:38
@solar3070 solar3070 deleted the feat/#180-meta-tag branch September 21, 2023 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

meta 태그 설정
4 participants