Skip to content
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

Feat/#50 category #53

Merged
merged 8 commits into from
Jul 16, 2022
Merged

Feat/#50 category #53

merged 8 commits into from
Jul 16, 2022

Conversation

NaveOWO
Copy link
Contributor

@NaveOWO NaveOWO commented Jul 15, 2022

πŸ“Œ λ‚΄μš©

  • μΉ΄ν…Œκ³ λ¦¬λ·° grid μ‚¬μš©ν•΄μ„œ κ΅¬ν˜„ν–ˆμŠ΅λ‹ˆλ‹€.

πŸ“Œ λ‚΄κ°€ μ•Œκ²Œ 된 λΆ€λΆ„

  • grid μ‚¬μš©λ²• μ•Œκ²Œ λμŠ΅λ‹ˆλ‹·
  • μ€„λ°”κΏˆ μ²˜λ¦¬ν•˜λŠ” pre-wrap μ•Œκ²Œ λμŠ΅λ‹ˆλ‹·
  • 곡톡 μ»΄ν¬λ„ŒνŠΈλ₯Ό λ³€ν˜•ν•΄μ„œ μ“Έ λ•Œ νƒ€μž…μ΄λ‚˜ ν”„λ‘­μŠ€λ₯Ό μ–΄λ–»κ²Œ μ£ΌλŠ”μ§€ μ•Œκ²Œ λμŠ΅λ‹ˆλ‹·

πŸ“Œ μ§ˆλ¬Έν•  λΆ€λΆ„

  • μ΄λ²ˆμ—” μ—†μ–΄μš©!! ν•΄κ²° μ™„λ£Œ!!
  • μ•„ ν•˜λ‚˜ μžˆλŠ”λ°, \n ν–ˆμ„ λ•Œ μ™œ μ€„λ°”κΏˆμ΄ μ•ˆλμ„κΉŒμš€....

πŸ“Έ μŠ€ν¬λ¦°μƒ·

Copy link
Contributor

@henization henization left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

μ§€μ˜ λ„˜ μ•Όλ¬΄μ§€κ²Œ 객체 잘 λ§Œλ“œλŸ¬λ”° 짱 !!!!!!!!!!

Comment on lines +4 to +15
export default function CategoryContents() {
type gridValueProps = {
id: number;
columnStart: number;
columnEnd: number;
rowStart: number;
rowEnd: number;
};
const gridValue: gridValueProps[] = [
{
id: 1,
columnStart: 1,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typeμ΄λž‘ const export default function μœ„μ—λ‹€κ°€ μ μ–΄μ€˜λ„ 될 κ±° κ°™μ•„μš©

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

μ•„ν•˜ν•˜ν•«!! κ·Έμƒˆ κΉŒλ¬΅μ—ˆλ‹Ή κ°μŸˆν•΄μš€ ν—€ν—€

Comment on lines +22 to +28
id: 2,
columnStart: 1,
columnEnd: 2,
rowStart: 6,
rowEnd: 9,
},

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

μ•Όλ¬΄μ§€κ²Œ 쀬넀 ~~ 짱짱!!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ν—Ών—€γ…”γ……!!

grid-gap: 0.8rem;
`,

CategoryItem: styled.article<{ columnStart: number; columnEnd: number; rowStart: number; rowEnd: number }>`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good~~

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

πŸ’™

@@ -22,6 +22,8 @@ export const St = {
Content: styled.p`
margin-top: 0.4rem;

white-space: pre-wrap;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

λΏŒλ“― γ…Žγ…Ž

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

졝였πŸ”₯πŸ”₯

Copy link
Contributor

@NYeonK NYeonK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

κ·Έλ¦¬λ“œ μž₯인 μœ€μ§€μ˜~!~!! 고생해쒀
μ €κ±Έ μ–΄λ–»κ²Œ ν•˜λ‚˜ μ‹Άμ—ˆλŠ”λ°, μ„±κ³΅ν•˜λ‹€λ‹ˆ.. λŒ€λ‹¨ν•΄μ–ΈλƒπŸ€—πŸ€—

Comment on lines +1 to 2
export { default as CategoryBanner } from "./categoryBanner.png";
export { ReactComponent as MoodPiickle1 } from "./MoodPiickle1.svg";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

밑에 μ“΄ κ±° λ‚΄κ°€ 썼던 κ±΄λŒœγ…Žγ…Ž defaultλž‘ ReactComponent의 차이가 뭘까?!? κΆκΈˆν•©λ‹ˆλ‹Ή!!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NYeonK
λ„ˆμ˜€γ…†κ΅¬λ‚˜?γ…Žγ…Žγ…Ž

svg λŠ” μ»΄ν¬λ„ŒνŠΈμ²˜λŸΌ μ‚¬μš©ν•˜κΈ° μœ„ν•΄μ„œ ReactComponent 둜 export μ‹œμΌœμ£Όκ³ ,
image (png) λŠ” src 값을 string 으둜 λ°›μ•„μ™€μ„œ img νƒœκ·Έμ˜ src 속성에 λ„£μ–΄μ£Όκ±°λ“ γ„΄ κ·Έλž˜μ„œ κ·Έλƒ₯ default 둜 export μ‹œμΌœμ€˜

❗ κ·Έλž˜μ„œ 말이지, μ•žμœΌλ‘œ svg νŒŒμΌμ€ asset/image κ°€ μ•„λ‹ˆλΌ asset/icon νŒŒμΌμ— λ„£μ–΄μ£Όμ„Έμš” !

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹ μ•„κ·ΈλŸ°κ±°μ—¬μ¨??
λ‚˜λŠ” μ €κ±° icon/index.tsx에 μžˆλŠ”κ±° κ³ λŒ€λ‘œ λ³΅λΆ™ν•΄μ™”μ§€γ…Žγ…Ž

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

λ„Ή~~~~~~~~~~~~

Comment on lines +13 to +14
grid-template-columns: repeat(2, 1fr);
grid-template-rows: repeat(14, 1fr);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

μ˜€μ™€ 이게 κ·Έλ¦¬λ“œκ΅¬λ‚­!!
repeatμ•ˆμ— 2, 1fr 이게 μ–΄λ–€ μ˜λ―ΈμΈμ§€ κΆκΈˆν•©λ‹ˆλ‹€!!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

repeat ν•˜λ©΄ μ»¨ν…Œμ΄λ„ˆλ₯Ό <repeat(λͺ‡μΉΈμœΌλ‘œ λ‚˜λˆŒκ»€μ§€, μΉΈλ“€μ˜ λΉ„μœ¨)> μ΄λ ‡κ²Œ μ„€μ •ν•  수 μžˆμ–΄μš§!!! 각각 ν–‰μ΄λž‘ 열을 μ—¬λŸ¬κ°œμ˜ λ‹¨μœ„λ‘œ λ‚˜λˆ μ£Όκ³  κ·Έ μ•ˆμ— μ•„μ΄ν…œλ“€μ΄ μ–Όλ§ŒνΌ μ°¨μ§€ν•˜λŠ”μ§€ κ³„μ‚°ν•΄μ„œ λ‚˜νƒ€λ‚΄μ£Όλ©΄ λ˜λ”λΌκΆ!!

grid-row-end: ${({ rowEnd }) => rowEnd};
background-color: white;
border-radius: 1.6rem;
border: 0.1rem solid #e0e0e0;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#e0e0e0 이 λΆ€λΆ„ theme에 gray200으둜 있던데, themeμ—μ„œ κ°€μ Έμ˜€λŠ” 건 μ–΄λ–¨κΉ‘?-?

Copy link
Member

@joohaem joohaem Jul 16, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

background-color 도 theme.colors.white 둜 κ³ μ³μ£Όμ„Έμš”-

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ν—€ν—€.. border 에 색깔도 theme μ¨μ•Όλ˜μ§€.. κΉŒλ­‡λ”°
background-color도 λ°”κΏ”λ†“μ„κ»˜μš€ κ°μŸˆν•΄μš€~~~~~

Copy link
Member

@joohaem joohaem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

πŸ‘πŸ‘πŸ”₯πŸŽ‡πŸŽ‡πŸŽ‡πŸ‰πŸ“·πŸ“ΈπŸ“·πŸ“ΈπŸ“·πŸ“Έ

Comment on lines +18 to +22
CategoryItem: styled.article<{ columnStart: number; columnEnd: number; rowStart: number; rowEnd: number }>`
grid-column-start: ${({ columnStart }) => columnStart};
grid-column-end: ${({ columnEnd }) => columnEnd};
grid-row-start: ${({ rowStart }) => rowStart};
grid-row-end: ${({ rowEnd }) => rowEnd};
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

옹 μ‹ κΈ°ν•΄ λŒ€λ°•

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

였예예~~~~

Comment on lines +7 to +15
export const categoryList: categoryListProps[] = [
{
id: 0,
title: "μ†Œκ°œνŒ…",
content: "μ»€ν”Œκ³Όμ˜ 좔얡을 κ³΅μœ ν•˜μ„Έμš”",
},
{
id: 0,
title: "μ†Œκ°œνŒ…",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

더미 객체이고, ν•˜λ£¨λ§Œμ— μˆ˜μ •λ  μ‚¬ν•­μ΄μ§€λ§Œ, id값은 λͺ¨λ‘ λ‹€λ₯΄κ²Œ μ„ μ–Έν•˜λŠ” 게 μ’‹λ‹€κ³  μƒκ°λ“­λ‹ˆλ‹€~>~

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아이디값을 μ•ˆλ°”κΏ¨λ„΄....!? μ•žμœΌλ‘œ 더 κΌΌκ³ ν•˜κ²Œ 체크 ν• κ»˜μš€!!! 감쟈감쟈

Comment on lines 42 to 68
CategoryItem2: styled.article`
grid-column-start: 1;
grid-column-end: 2;
grid-row-start: 6;
grid-row-end: 9;
background-color: white;
border-radius: 1.6rem;
border: 0.1rem solid #e0e0e0;
`,
CategoryItem3: styled.article`
grid-column-start: 1;
grid-column-end: 2;
grid-row-start: 9;
grid-row-end: 12;
background-color: white;
border-radius: 1.6rem;
border: 0.1rem solid #e0e0e0;
`,
CategoryItem4: styled.article`
grid-column-start: 1;
grid-column-end: 2;
grid-row-start: 12;
grid-row-end: 15;
background-color: white;
border-radius: 1.6rem;
border: 0.1rem solid #e0e0e0;
`,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…Œγ…‹μ• μžμΌ 쑍 λ€Ό

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹γ…‹μ•„λœ¨ μ‹Ήλ“€μΌ°λ„€

@joohaem joohaem merged commit a625dcd into develop Jul 16, 2022
@joohaem joohaem deleted the feat/#50-Category branch July 16, 2022 05:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Category - MoodPiickle ] μΉ΄ν…Œκ³ λ¦¬λ·° κ΅¬ν˜„
4 participants