Skip to content

fix: header 시멘틱 태그 변경, div 삭제에 따른 전반적인 코드 수정#268

Merged
junjeeong merged 3 commits intodevfrom
chore/header-부분-시멘틱-태그-수정
May 16, 2025

Hidden character warning

The head ref may contain hidden characters: "chore/header-\ubd80\ubd84-\uc2dc\uba58\ud2f1-\ud0dc\uadf8-\uc218\uc815"
Merged

fix: header 시멘틱 태그 변경, div 삭제에 따른 전반적인 코드 수정#268
junjeeong merged 3 commits intodevfrom
chore/header-부분-시멘틱-태그-수정

Conversation

@junjeeong
Copy link
Collaborator

@junjeeong junjeeong commented May 11, 2025

Pull requests

✅ 작업한 내용

  • Header.tsx에 margin-top 14px을 주던 div를 삭제했고, 이를 index.css에 main에 주었습니다.
  • 모든 페이지에서 Header 아래로 오는 부분은 본문 영역으로 판단해, main 태그로 감싸주었습니다.
  • Header.tsx에 가장 바깥 div를 header로 바꾸고 안에 있는 MainHeader, SubHeader를 div로 바꾸었습니다.
  • 몇개의 페이지에서 중복되는 레이아웃 div가 있어 삭제하고 ContentLayout 컴포넌트에 자식 div에 style을 조금 수정했습니다. 수정 이후에도 기존과 동일한 ui로 보이는지까지 모두 확인했습니다.
  • Chat 페이지에서 div를 section으로 나누는게 좋을 것 같아 section으로 시멘틱 태그를 부여해주었습니다.

🔢 #267

❗️PR Point

  • 없음

📸 스크린샷

@junjeeong junjeeong requested a review from Soohyuniii May 11, 2025 00:05
@junjeeong junjeeong self-assigned this May 11, 2025
@junjeeong junjeeong added ⚙️ Chore 소규모 수정 🎨 Style 스타일 수정 labels May 11, 2025
@junjeeong junjeeong linked an issue May 11, 2025 that may be closed by this pull request
3 tasks
@junjeeong junjeeong changed the title chore: header 시멘틱 태그 변경, div 삭제에 따른 전반적인 코드 수정 fix: header 시멘틱 태그 변경, div 삭제에 따른 전반적인 코드 수정 May 11, 2025
@junjeeong junjeeong added 🛠️ Fix 수정 and removed ⚙️ Chore 소규모 수정 labels May 11, 2025
@Soohyuniii
Copy link
Collaborator

로그인 옆 패딩이 너무 부족한 것 같아요!
image

@Soohyuniii
Copy link
Collaborator

가운데 정렬도 안되어있어요!
image
최대한 pr 재성성없이 해당 pr에 수정 부탁드립니다

@junjeeong junjeeong closed this May 14, 2025
@junjeeong
Copy link
Collaborator Author

네 고치겠습니다!

@junjeeong junjeeong reopened this May 14, 2025
@Soohyuniii
Copy link
Collaborator

width가 500px보다 작은 경우 아래와 같이 화면이 잘리는 현상 발생합니다.
image

@junjeeong
Copy link
Collaborator Author

616767d

수정했습니다!

@junjeeong junjeeong merged commit c7f0df1 into dev May 16, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠️ Fix 수정 🎨 Style 스타일 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chore: Header 부분 시멘틱 태그 수정

2 participants