Skip to content

Taebin week2 task #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.

Sign up for GitHub

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

Jump to bottom
Open
seulnan wants to merge 50 commits into
base: main
Choose a base branch
from
Open

Taebin week2 task #7

seulnan wants to merge 50 commits into from

Conversation

@seulnan
Copy link
Owner

@seulnan seulnan commented Sep 29, 2024 •
edited
Loading

전체적인 피드백

  1. 일단 압도적으로 매우매우 높은 완성도..별점 오만개.
  2. 파일이름도 너무 잘 잡았어용! 다만, 파일구조면에서 폰트나 이미지는 assets폴더에 따로 저장해두는 경우가 많아용
  3. css파일은 따로 분리해봅시다!

layout

  1. 클래스 이름 pur(ple)_box이런식으로 짓는 것도 좋지만 다른 프론트, 백이 이해가 잘되려면 더 직관적으로 그 내용이 뭔지를 중심으로 짓는게 좋겠죠? ex) review_box
  2. p_info로 p_name&p_des묶어서 레이아웃 조절한거 너무 굿굿! block속성을 잘 이용했어요!
  3. layout 배경이미지는 어디에...?있나요...?

recipe

  1. ingredients&nutrition부분 여백이 왤케 크죠...?디자인대로 수정부탁드려용! 아마 section-nutrtion의 gap을 조정하면 될것같긴한데...
  2. ing_test에서 p태그와 display를 block으로 설정했던데 p태그는 기본값이 인라인+ block 따로 설정하기보다 그냥 바로 block인 div를 이용하는건 어떨까용? 참고로 svg태그는 기본속성이 인라인입니다!
  3. 전체적으로 목록태그랑 content가 높이가 안맞는것같습니다 p태그를 이용해서 1. 2. 3. 이런식으로 짰던데 이렇게 간단한 커스텀을 요구하는 사이트는 ul, ol, li태그를 적극 이용해봅시다!
  4. 위내용에 살짝 원인인 부분이 span태그를 이용했던데 flex에서는 div태그가 좀더 유연하게 대처가능해서 자주 씁니다! span은 아이콘을 감쌀때나 특정 텍스트를 강조할때(형광펜치듯이) Style을 지정하고싶을 때 주로 사용합니다!

@seulnan seulnan changed the title Taebin Taebin week2 task Sep 29, 2024
gaeun2o added a commit that referenced this pull request Oct 30, 2024
@gaeun2o
[fix] layout : 여백, 정렬, 파일 위치 수정
4207fdd 
코드리뷰 바탕으로 코드를 수정했습니다.
1. 이미지 파일들 assets 파일에 넣기
2. class b_text 양쪽 여백 맞추기
3. h1, h2 마진값 수정
4. background 가로 세로 vw, vh 단위로 변경

Resolves:#7
jangtaebin3 and others added 23 commits November 6, 2024 02:07
@jangtaebin3
[Feat] 미디어 쿼리 1차 적용
334c122 
추가 수정 필요
@jangtaebin3
Revert "[Feat] 미디어 쿼리 1차 적용"
736b904 
This reverts commit 334c122.
@jangtaebin3
[Feat] 미디어 쿼리 1차 적용 완료
5d086d0 
추가적인 수정 필요
@jangtaebin3
Revert "[Feat] 미디어 쿼리 1차 적용 완료"
77724ac 
This reverts commit 5d086d0.
@jangtaebin3
[Feat] 미디어 쿼리 기능 적용
1727794 
일부 수정 필요
@jangtaebin3
[Fix] 미디어 쿼리 적용 완료
f8ee885 
review_box 너비 문제 해결
people_box 너비/여백 문제 해결
@jangtaebin3
[Fix] Recipe 페이지 완성
5dc1fab 
줄 간격 오류 수정
@jangtaebin3
[Feat] todo_app 1차 위치 배열, 기능 구현
4767fc1 
위치, 기능 구현
다크 모드, 폰트 적용, 버튼 디자인 등 수정 필요
@jangtaebin3
[Fix] 이미지 적용, 폰트 적용 완료
f281e2f 
하단 버튼 구조 수정 필요
@jangtaebin3
[Fix] 버튼 구성 요소 수정 완료, todo app 완성
11ceeb3
@jangtaebin3
[Fix] 구분선 색상 적용, 입력창 테두리 수정
046ea4b
@jangtaebin3
[Fix] 다크 모드 체크 박스 hover 상태 수정
0994d9f
@jangtaebin3
리액트 훅 실습 코드
5225bde
@jangtaebin3
리액트 훅 예제 실습
2615353
@jangtaebin3
Delete mistake
3fe8404 
리액트 초기 세팅 미숙
@jangtaebin3
create ignore
8e1d899
@jangtaebin3
tic-tac-toe 기능 구현
f0099f1 
새로고침 기능, 다음 게임 기능 수정 필요
@jangtaebin3
파일 구조 조정
e79dda0
@jangtaebin3
파일 이름 변경
f58bd8a
@jangtaebin3
gitignore .yarn 무시 추가
1c78261
@jangtaebin3
gitignore .yarn 무시 추가
d0bd90c
@jangtaebin3
파일 구조 조정
309bfdd
@jangtaebin3
Remove .yarn folder from repository
3110aa6
@jangtaebin3
Remove node_modules from Git
cbc8b05
@jangtaebin3
Remove __MACOSX and _tic-tac-toe from repository
89ca525
@jangtaebin3
Update gitignore
ade7a81
@jangtaebin3
[Add] StartPage 디자인 요소 추가
349214d
@jangtaebin3
assets 폴더 추가
b05f410
@jangtaebin3
[Fix] StartPage 디자인 세부 조정
db72adf
@jangtaebin3
[Fix] 칸 크기, 칸 간격 조정
cf9a823
@jangtaebin3
[Add] 게임 페이지 상단 요소 추가
3123489 
로고, X TURN, O TURN, 새로고침 버튼 추가
@jangtaebin3
[Fix] GamePage 상단 요소 위치 조정
4b3b423
@jangtaebin3
[Add] 폰트 적용
9f97d79
@jangtaebin3
[Fix] 게임보드 그림자 추가
1935e0c
@jangtaebin3
[Fix] 새로고침 기능 수정, 새로고침 hover 상태 추가
159b589
@jangtaebin3
[Add] Modal.js 디자인 적용
7ef7a61
@jangtaebin3
[Fix] 폰트 사이즈 수정
84c8d83
@jangtaebin3
파일 구조 정리
b920e02
@jangtaebin3
[Fix] X 선택 버튼 수정
60d98da
@jangtaebin3
[Fix] X 버튼 수정
fcd4f70
@jangtaebin3
[Fix] 일부 경우 점수판 조정
7f8f828
@jangtaebin3
[Add] yarn install
fdebbce
@jangtaebin3
[Fix] 선택 상황에 따른 O 버튼 수정
51e9df9 
선택 상황에 따라 O 버튼의 색이 변하는 것까지는 구현되었지만 색이 정확하지 않음
@jangtaebin3
[Fix] O 버튼 기능 수정
9ee9bb6 
O / X 를 선택했을 때의 상태를 수정함
@jangtaebin3
[Add] RESTART Modal 추가
44aa3fc 
RESTART Modal 추가 완료
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@seulnan @gaeun2o @jangtaebin3