Skip to content

Gaeun week2 task#5

Open
seulnan wants to merge 27 commits intomainfrom
Gaeun
Open

Gaeun week2 task#5
seulnan wants to merge 27 commits intomainfrom
Gaeun

Conversation

@seulnan
Copy link
Owner

@seulnan seulnan commented Sep 27, 2024

전체적인 피드백

  1. 일단 굉장히 완성도 높은 사이트라 칭찬합니다!!!너무너무 고생많았어용
  2. 가장 칭찬하고싶은점은 작은단위로 커밋한것과 커밋컨벤션 지킨거 최고.
  3. 파일구조는 보통 assets(폰트, 이미지)폴더로 분리해서 사용합니다!
  4. 부모태그에 relative를 잡고 그 아래 자식태그들을 absolute로 설정하는것은 확실히 구현할 수 있는 대신 반응형디자인할때 매우많이 깨질거라는 점.....을 고려해서 반응형디자인때 레이아웃이 바뀌는 부분"만" flex로 바꿔봅시다! 또 다 flex로 바꿀 이유는 업서요! relative&absolute가 확실한건 맞으니까!

layout

  1. class b_test 양쪽여백이 안맞는 점 수정부탁드려용
  2. h1, h2 태그 마진값이 너무 작은것같은데 확인부탁드립니다

recipe

  1. nutrition 속 item들 위치 디자인대로 조절해주세용
  2. 그외 Bullet 속 item들,,,위치 수정필요합니다

gaeun2o and others added 27 commits September 25, 2024 17:43
Feat : 배경 및 요소 추가

layout.html : 텍스트 정렬을 제외한 배경, 도형 추가, 사진 삽입
recipe.html :  사진, 도형 삽입을 통해 기본적인 틀 구성

Resolves: #1
Feat : bold 및 폰트,pink box 위치 조정 추가

기본 틀에서 title에 bold 추가,
색상 지정,
pink box를 중앙으로 위치 조정 했습니다.

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

사진 및 텍스트 추가, 위치 조정해서 마무리 했습니다

Resolves : #3
Feat : text 속성, 기능 추가 및 변경

폰트 변경, bold, 색상 변경, text 앞 요소 추가했습니다.
Nutrition 아래 content 위치 정렬과 class 가독성 있게 이름 정리할 예정입니다.

Resolves: #4
Style : class 이름 변경

이해하기 쉽도록 class의 이름을 변경했습니다.

Resolves : #5
Style : recipe - class 이름 변경

class의 이름을 변경하여 가독성을 높였습니다. Nutrition 아래 text box의 정렬 수정이 필요합니다..

Resolves:#6
미디어 쿼리 실습 코드 제출합니다
코드리뷰 바탕으로 코드를 수정했습니다.
1. 이미지 파일들 assets 파일에 넣기
2. class b_text 양쪽 여백 맞추기
3. h1, h2 마진값 수정
4. background 가로 세로 vw, vh 단위로 변경

Resolves:#7
[fix] flex로 수정

반응형을 사용할 때 레이아웃이 바뀌는 요소들을 flex로 수정했습니다.

resolves : #8
[feat] 반응형 기능 추가 - 미디어쿼리

layout.css 에 미디어쿼리를 활용하여 모바일 크기일 때 사용할 수 있는 반응형 기능을 추가했습니다.

resolves : #9
[fix] recipe : 코드리뷰 반영 및 수정

- nutrition, bullet 속 item 위치 및 디자인 수정
- 폰트 통일
- container 안 max-width 사용
- 원 아이콘 svg 대신 radius 이용하도록 수정

resolves : #10
[fix] 사진 깨짐 수정 + 코드 분리

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

resolves : #11
[fix] 겹침, 박스 길이 조정

box_s의 겹침 현상을 해결하고,
box_B 의 height,width를 각각 vh,vw로 수정했습니다.

resolves : #12
[fix] icon 크기 축소 및 위치 조정

icon 크기를 줄이고 위치 조정했습니다

resolves : #13
[fix] box_s 위치 조정

box_s의 위치를 시안과 동일하도록 조정했습니다.

resolves : #14
[Feat] todo 기능구현

아직 스타일은 적용하지 못한 정말 기능만 구현된 ... todo입니다.
할 일 추가, hover 했을 때 기능들 , 테마 변경 , AII / Completed / Active필터와 Clear completed기능 , 남은 할 일 개수 기능이 있습니다.

resolves : #15
[fix] 스타일 변경

피그마 시안과 비슷하게 위치를 조정하고, 세부 스타일을 변경했습니다.
앞으로 타이핑 박스 테두리 색상과 체크 박스 디자인을 변경할 예정입니다.

resolves : #16
[feat]  react  실습코드 제출-1

 useState  - 실습코드 1 입니다.
[fix] 스크롤 기능 수정,checkbox 스타일 변경

스크롤 기능을 추가하고 checkbox 스타일을 시안과 동일하게 변경했습니다.

resolves : #17
[fix] 디자인, 기능 수정

1. clear completed 누르면 다 삭제되는 오류를 completed 된 항목만 지워지도록 data-filter="completed" 추가해서 수정하였습니다.

2.  #input-text color : #393A4B;로 수정하였습니다.
3. 개수 넘어가면 task 박스 길이도 늘어나게 max-height를 삭제하여 해결했습니다.
4. 처음 열었을 때 button data-filter="all"이 적용되고 있는데도 class="filter-button hover active"가 적용되어있지 않아 적용시켰습니다.
처음 설정을 아래 처럼 <button class="filter-button hover active" data-filter="all">All</button> 이렇게 해결했습니다.

resolves:#18
[fix] dark 그림자, clear 디자인 수정

dark mode 일때 box-shadow를 설정하고 clear filter button에 hover 했을 때의 기능을 추가했습니다.

resolves : #19
로컬스토리지 기능 작동하도록 수정했습니다. 

resolves : #20
[fix]로컬스토리지_파일구조수정
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