Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 12 additions & 19 deletions pr01/data/김희성.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
export const 김희성 = [];

김희성.push({
name: "최은비",
name: "김희성",
target: "2팀",
title: "refactor : 1팀 김희성 → 신진호 #69",
refactors: [
Expand All @@ -10,17 +10,15 @@ export const 김희성 = [];
prNumber: 69,
details: [
{
detail: `<br>
1. 메인 메뉴들 font-size 16px → 20px 변경 <br>
2. 메인 메뉴들 text-shadow 추가
`,
intention: `<br>
1. 메인 메뉴의 글씨크기를 늘려 차별성 두기 위해 추가<br>
2. 전체적으로 심플한 스타일이라서 shadow추가로 메뉴에 시선이 가게하기 위해 추가
`
},
detail: "font-size 16px → 20px 변경",
intention: "메인 메뉴들 text-shadow 추가"
},
{
detail: "메인 메뉴의 글씨크기를 늘려 차별성 두기 위해 추가",
intention: "전체적으로 심플한 스타일이라서 shadow추가로 메뉴에 시선이 가게하기 위해 추가"
},
]
}
},
]
});

Expand All @@ -34,16 +32,11 @@ export const 김희성 = [];
prNumber: 46,
details: [
{
detail: `<br>
1. js의 애니매이션 배경 추가<br>

`,
intention: `<br>
1. 전체적으로 무난하여 배경을 추가
`
detail: "js의 애니매이션 배경 추가",
intention: "전체적으로 무난하여 배경을 추가"
},
]
}
},
]
});

53 changes: 26 additions & 27 deletions pr01/data/최은비.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,16 +10,15 @@ export const 최은비 = [];
prNumber: 48,
details: [
{
detail: `<br>
1. grid-template → grid-template-columns 옵션으로 변경<br>
2. 링크가 연결된 글씨에 hover를 추가
`,
intention: `<br>
1. grid-template-columns 옵션으로 열만 설정하면 되서 가독성 향상<br>
2. 링크가 연결됨을 마우스 hover를 통해 다른 요소들과 구분
`
detail: "grid-template → grid-template-columns 옵션으로 변경",
intention: "가독성 향상",
},
]
{
detail: "링크가 연결된 글씨에 hover를 추가",
intention:"링크가 연결됨을 마우스 hover를 통해 다른 요소들과 구분",
},
],

}
]
});
Expand All @@ -34,15 +33,14 @@ export const 최은비 = [];
prNumber: 76,
details: [
{
detail: `<br>
1. fav-music 클래스 생성하여 노래 제목에 링크 삽입<br>
2. movie-container 클래스 생성해서 영화 티저 영상 삽입
`,
intention: `<br>
좋아하는 음악과 영화를 링크나 영상으로 공유하면 좋을 것 같아서
`
detail: "fav-music 클래스 생성하여 노래 제목에 링크 삽입",
intention: "링크가 있으면 좋을 것 같아서",
},
{
detail: "movie-container 클래스 생성해서 영화 티저 영상 삽입",
intention:"iframe으로 영상 달면 좋을 것 같아서",
},
]
],
}
]
});
Expand All @@ -57,17 +55,18 @@ export const 최은비 = [];
prNumber: 79,
details: [
{
detail: `<br>
1. css 파일에서 alert를 띄우는 편지모양 아이콘 버튼과 좋아하는 노래 제목에 마우스 커서 포인터 설정<br>
2. 노래 제목 클릭해도 링크로 넘어가도록 HTML 파일 내 div 위치 수정하고 fav-music 클래스 생성하여 css파일에서 커서 포인터 옵션 설정<br>
3. 페이지 내 마우스 우클릭 시 alert 창에 뜨는 문구 "좌클릭 금지"-> "우클릭 금지"로 문구 변경<br>
`,
intention: `<br>
1. 클릭시 alert나 링크 연동이 되어있는 요소에 마우스 포인터를 통해 다른 요소들과 구분<br>
2. 노래 제목도 클릭시 링크로 연결되면 좋을것 같아서
`
detail: "css 파일에서 alert를 띄우는 편지모양 아이콘 버튼과 좋아하는 노래 제목에 마우스 커서 포인터 설정",
intention: "클릭시 alert나 링크 연동이 되어있는 요소에 마우스 포인터를 통해 다른 요소들과 구분",
},
]
{
detail: "노래 제목 클릭해도 링크로 넘어가도록 HTML 파일 내 div 위치 수정하고 fav-music 클래스 생성하여 css파일에서 커서 포인터 옵션 설정",
intention:"노래 제목도 클릭시 링크로 연결되면 좋을것 같아서",
},
{
detail: "페이지 내 마우스 우클릭 시 alert 창에 뜨는 문구 '좌클릭 금지'-> '우클릭 금지'로 문구 변경",
intention: "오타 수정(일부러 좌클릭 금지로 하신걸까요??)"
}
],
}
]
});