-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
[ 1주차 기본/심화 과제 ] 가계부 💸 #2
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋은 코드 많이 얻어갑니다롱😆
} | ||
|
||
/*전역 적용*/ | ||
.income { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저는 리스트랑 자산부분의 수입지출을 따로 스타일입혔는데 같이 입히는게 더 좋네요!!!
font-size: 1.4rem; | ||
} | ||
|
||
.detail-amount__plus, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
class명을 보고 어떤 역할을 하는건지 알 수 있어서 가독성이 좋네요!! 다음과제부턴 저도 보내준 링크 보고 참고해야겠어요ㅎㅎㅎ
gap: 1rem; | ||
} | ||
|
||
/* checkbox input 커스텀 */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checkbox 수입/지출 나눠서 하는 섬세함..👍🏻 미리 기능을 생각하고 스타일을 입히는게 중요하다는걸 깨닫고 갑니다!!ㅎㅎ
|
||
/*리스트 내 세로 스크롤*/ | ||
.history__list { | ||
height: 42vh; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
height값을 값으로 지정해주니 디바이스 높이에 따라 여백이 남을 때가 있더라구요! 그래서 저는 cal()함수 이용해서 상단부분과 하단부분의 정적인 높이값을 빼주는 방법을 사용했습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오오 맞아요!! 저는 처음에 이 부분 놓치고 급하게 구현하다 보니까 vh로 임시로 맞춰줬는데 calc를 사용하는 방법으로 리팩토링 해볼게요 ><
font-size: 1.1rem; | ||
} | ||
|
||
/*수입, 지출에 따라 부호 붙여주기*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
css로 수입,지출에 따라 부호 붙이는방법이 있군요!!!
week1/assign4/index.html
Outdated
|
||
<section class="history"> | ||
<article class="history__top"> | ||
<button><</button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<,>는 예약문자이기 때문에 <,>를 사용해야하는군요!!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
맞습니당 !!! 예약 문자는 요기 참고해보면 좋을거 같아요 ~
} | ||
|
||
.history-category { | ||
width: fit-content; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
width:fit-content
를 해도 디바이스의 너비가 줄어들면 text가 가로로 정렬되네요 history-contents의 너비가 8.5rem으로 정해져서 그런건가요???
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 맞습니당 ! 가로로 정렬되는 영역만큼만 반응형 되도록 history-contents의 너비를 지정했어용
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생했어용 ~~ 폰트 귀엽다 ㅎㅎ
border-width: 0.2rem 0; | ||
} | ||
|
||
.asset__box { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
연서가 알려준 BEM 방식 ,, 바닐라로 코딩할때는 진짜진짜 유용한것같아 !! 나도 한번 익혀봐야겠어 이번 과제할때 확실히 필요성을 느꼈어 ㅎㅎ
} | ||
|
||
/*전역 적용*/ | ||
.income { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
마자 나도 이거 나중에 앗차 글케할걸 싶었어
|
||
/*수입, 지출에 따라 부호 붙여주기*/ | ||
.history__list__box > .income::before { | ||
content: "+"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
우와 이거 약간 짬이 느껴진다 ,, 난 냅다 div로 + 감싼거 같은디 ㅎ,ㅎ 이거 보면서 방금 찾아봤는데 content 가상속성 사용하면 되게 js처럼 이것저것 할 수 있는게 많네 !! 신기하다 덕분에 하나 배워가용 ~
/* font: inherit; */ | ||
vertical-align: baseline; | ||
} | ||
/* HTML5 display-role reset for older browsers */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혹시 이 파일은 어떤거야?? 다 초기값으로 세팅하려는건가??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
맞습니당 ! 브라우저로부터 상속받는 기본 스타일링을 미리 다 제거해서, 의도치 않은 스타일링 결과가 나오지 않게 해주는 파일이에용!
요기를 참고하면 더 좋을거 같아용 !!
|
||
/* 전체 스크롤 방지 */ | ||
overflow: hidden; | ||
touch-action: none; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오오 이거 첨알았어
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
캬아.. 디테일 미쳤다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이게 OB다,, 신경 많이 쓴게 느껴진다 !! 심화과제까지 고생 많았어용 ✨
<section class="asset"> | ||
<article class="asset__box"> | ||
<h2 class="asset__box__title">나의 자산</h2> | ||
<p class="asset__box__total-amount">💲249700</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
동적으로 값을 바꿀 때는 css에서 before로 $ 붙여주는 것도 좋을 것 같네욤!ㅎㅎ
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오오 맞습니당 요것도 before 속성 사용하는게 좋을것 같네유 바로 수정 !!!
week1/assign4/index.html
Outdated
<span class="history-category">식비</span> | ||
<span class="history-contents">맥도날드 안암역점</span> | ||
<span class="history-amount expense">10800</span> | ||
<button class="history-del-btn">X</button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
button 태그는 reset, button, submit 세가지의 type을 가져요!
default는 submit이므로, 이렇게 type을 button으로 명시해주는 게 좋답니당!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
앗 맞아요 ㅎㅎ 깜빡한 부분인데 저도 코드리뷰 하다가 생각났네용 바로 수정 갈기겠습니댜 🖤
|
||
/* 전체 스크롤 방지 */ | ||
overflow: hidden; | ||
touch-action: none; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
캬아.. 디테일 미쳤다
✨ 구현 기능 명세
⭐️기본 과제
1. 제목
2. 나의 자산
3. 날짜
4. 수입, 지출 선택
5. 내역 리스트
flex
로 배치합니다.카테고리
내역 내용
지출 및 수입 금액
삭제 버튼 x
가 있습니다.flex
로 배치합니다. (선택)6. 리스트 추가 버튼
⭐️ 심화 과제
1. checkbox input 커스텀
2. 내역 리스트
내역 내용
이 일정 길이보다 길어질 경우...
로 말줄임 처리됩니다.3. 반응형
💎 PR Point
1️⃣ 나의 자산 영역
2️⃣ 수입, 지출 선택
3️⃣ 내역 리스트
4️⃣ 리스트 추가 버튼
🥺 소요 시간, 어려웠던 점
3h
🌈 구현 결과물
노션 링크로 첨부합니댜 ~
https://quasar-smelt-153.notion.site/week1-assign4-cf8346270f3c42fe82b9f22c08ed789e?pvs=4