Skip to content

Comments

Sohyun week2 task#11

Open
seulnan wants to merge 21 commits intomainfrom
Sohyun
Open

Sohyun week2 task#11
seulnan wants to merge 21 commits intomainfrom
Sohyun

Conversation

@seulnan
Copy link
Owner

@seulnan seulnan commented Sep 29, 2024

전체적인 피드백

  1. 폴더구조를 recipe처럼 layout도 assets폴더에 이미지, 폰트 따로 분리해봅시다!
  2. css파일도 분리해줘야 좋아용!!
  3. 완성도 1등. 진짜 최고. 너무너무 고생많았어용

layout

  1. absolute를 많이 썼던데 이제 반응형할때 좀 힘들어질수도있으니 div&flex적극이용합시다
  2. background-image도 너무 잘넣었어요!!

recipe

  1. li(블록) & span(인라인)을 잘 이해하고 올바른 방법으로 활용해서 짠 코드입니다! 최고!
  2. nutrition 여백조금 줄여야할것같네용 원인은 좀 큰 단위의 부모태그 div.white-back에서 gap 을 너무 크게 설정한것같아요 부모태그에서는 상속되면 충돌이 많이 일어날 수 있으니 최대한 공통적인 속성만 설정하는것이 좋겠죠? 폰트라든지, 기본적인 글자색이라든지! 아니면 gap 말고 다른걸로 여백을 만들수도 있고!
  3. instruction 목록숫자 컬러 디자인대로 수정부탁드려용

Fix: 이미지 태그 중복 사용 제거

이미지 태그가 중복 사용되어 이미지가 겹쳐 보이는 문제가 발생하였습니다. 따라서 하나의 이미지 태그만 남기고 나머지 이미지 태그는 삭제하였습니다.

Resolves: #1
Feat: 나머지 기능 추가

Ingredients, Instruction, Nutrition 부분 추가 및 각 요소 구현

Resolves: #2
Fix: 숫자 버그 수정 및 간격 조절

Instructions 부분 목록에서 내용이 많아 다음 줄로 넘어가면 내용 시작 부분부터 시작되는 문제 발생
ol > li 사용하여 수정 후 간격 조절

Resolves: #3
Feat: 폰트 추가

Young Serif, Outfit 폰트 추가

Resolves: #4
Design: UI 디자인 구현

제목, 내용, 리뷰 기능 구현
Chat GPT 활용

Resolves: #5
Feat:폰트 추가

League Spartan 폰트 추가

Resolves: #5
Feat: recipe CSS 파일 분리

코드 리뷰를 반영하여 recipe의 HTML에 있던 style을 CSS 파일로 분리하였습니다.

Resolves: #12
Feat: layout CSS 파일 분리 및 image 파일 추가

layout의 HTML에 있던 style을 CSS 파일로 분리하였고, image 파일 추가하여 image를 image 파일에 추가하였습니다.

Resolves: #13
Fix: 전반적 코드 수정

absolute를 많이 사용하여 추후에 반응형 디자인 할 때를 위해 div & flex를 사용한 코드로 바꾸었습니다.

Resolves: #13
Fix: 여백 및 숫자 수정

부모 태그의 gap을 수정하고 spacing 클래스를 가진 div를 추가함으로써 전체적인 간격 및 nutrition 여백을 조정하였습니다.
instruction 목록 숫자에 컬러를 추가하였습니다.

Resolves: #14
미디어쿼리 실습 코드
Feat: layout 반응형 기능 추가

layout에 media-query를 이용하여 반응형 기능을 추가하였습니다.

Resolves: #15
Fix: 배경 수정

배경 수정

Resolves: #16
gaeun2o added a commit that referenced this pull request Nov 5, 2024
[fix] 사진 깨짐 수정 + 코드 분리

사진 파일을 새로 추가하고 css, html 로 파일을 분리했습니다.

resolves : #11
Feat: Todo 기능 구현

Todo 기능을 구현하였습니다
Fix: hover 기능 수정

여러 hover 기능 및 다른 기능들을 수정하였습니다.
Fix: cancel hover 수정

text에 hover하였을 때만 cancel이 보였는데 각 todo에 마우스가 hover 되면 cancel이 보이도록 수정하였습니다.

Resolves: #17
Fix: 로딩 시 oval 문제 해결

새로고침 시 light oval이 적용되는 문제가 발생하여 필터링 함수 부분에서 수정하여 해결하였습니다.

Resolves: #18
Fix: filterTodos 함수 부분...

아직...고치는중.........................
Fix: filterTodos 수정

filterTodos를 수정하여 oval이 제대로 보이도록 만들었습니다

Resolves: #19
Feat: 드래그 앤 드롭 다운 기능 추가

드래그 앤 드롭 다운으로 각 todo 위치를 변경 가능하게 하는 기능을 추가하였습니다.

Resolves: #20
React 실습 코드
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants