[#67] index.html 수정 등 단순 수정

[gummmmmy0v0]

📝 작업 내용

• index.html 수정
• 단순 수정

📷 스크린샷 (선택)

🧐 해결해야 하는 문제 (선택)

👀 새로 알게 된 내용 (선택)

💬 리뷰어에게 남길 말 (선택)

브랜치를 또 실수해서 반응형 커밋이 여기에도 포함되었어요...

[cskime]

커밋 정리하고 싶으시면 아래 순서대로 해 보시겠어요?

1. feature/#67 브랜치와 같은 위치에 아무 이름으로 브랜치 생성 (커밋 임시 보관용)
2. feature/#67 브랜치를 upstream:develop에 hard reset
   • 이 단계에서 upstream:develop이 최신 상태인지 확인해주세요.
   • GitKraken 사용하시면 브랜치 싱크가 안맞을 때가 가끔 생기는데요. 다른 탭 갔다오시면 싱크가 다시 맞춰집니다.
3. index.html 코드 개선 #67 작업에 해당하는 commit만 cherry pick, 또는 관련 작업을 다시 commit
4. origin:feature/#67로 force push

이런 작업을 할 때 commit에 issue 번호를 명시하는 규칙이 빛을 발합니다. 코드를 일일이 확인해 보지 않아도 어떤 commit을 cherry pick 하면 되는지 바로 알 수 있으니까요.

---

background-select.jsx 파일에서 다음 내용을 확인하고 수정해주세요.

1. 파일 안에서 prettier를 실행해서 코드 format을 맞춰주세요. VSCode에서 저장할 때마다 formatter를 실행하도록 설정해 두셨으면 cmd+S 단축키를 눌러서 실행시킬 수 있을거에요. 아마 import { media } ~ 위치가 변경될겁니다.
2. 38번 줄에 } 하나가 잘못 들어가있어서 error가 발생하고 있는 부분을 수정해주세요. 에러가 너무 명확한데, 아까 병합하고 전체 동작 확인할 때 문제가 없었던게 이상하네요.

파일마다 코드 수정하신 뒤에 꼭 두 가지를 확인해주세요.

1. Formatter가 실행되었는지
2. 자바스크립트 문법 오류 또는 ESLint 오류가 발생하지 않는지

---

send-message-page.jsx 파일에 DefaultAvatar가 두 번 선언되어서 발생하는 error를 수정해주세요.

---

index.html 수정 및 디폴트 아바타 찌그러짐 수정 commit에는 아래와 같은 문제가 있습니다.

1. Commit message 규칙이 지켜지지 않았습니다.
2. Commit message가 commit에 포함된 수정 내용을 모두 표현하지 못합니다. send-message-page.jsx의 변경 사항은 message에 명시하지 않을 만큼 작은 작업은 아닌거 같은데요. index.html을 수정하는 commit 1개와 send-message-page.jsx를 수정하는 commit 1개로 나눠도 좋을 것 같아요.

[cskime]

send-message-page.jsx 파일에 formatter가 적용되지 않은 것 같아요.
이 파일에서 formatter를 실행시키면 import 코드 순서가 바뀌어서 수정사항으로 잡히게 되는데요.
Formatter를 실행시킨 결과로 한번 더 commit하고 이 PR에 추가하거나, 다음 PR에서 수정된 내용을 반영해주세요.

[cskime]

발표 시연 준비를 위해 우선 병합하겠습니다. 수정이 필요한 부분은 다른 PR을 다시 만들겠습니다.