Skip to content

Commit

Permalink
Merge pull request #249 from BCSDLab/feat/#248/resolve-timetable-QA-i…
Browse files Browse the repository at this point in the history
…ssue

[시간표] 시간표 2차 스프린트 이슈 해결 (강의 정보, 타임테이블)
  • Loading branch information
hanagertrudeKim authored May 9, 2024
2 parents 1ca4f5d + a0c62b0 commit 4c5ade1
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 9 deletions.
5 changes: 3 additions & 2 deletions src/components/TimetablePage/LectureTable/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -62,14 +62,15 @@ function LectureTable({
key={header.key}
>
{header.label}
{headerIndex !== LECTURE_TABLE_HEADER.length - 1 && (
{/* 내림차순 기능 추가할때 다시 복구 */}
{/* {headerIndex !== LECTURE_TABLE_HEADER.length - 1 && (
<>
<button type="button" className={styles.table__button}>
<img src="https://static.koreatech.in/assets/img/ic-arrow-down.png" alt="내림차순" />
</button>
<div className={styles.table__resize} aria-hidden />
</>
)}
)} */}
</div>
))}
</div>
Expand Down
4 changes: 2 additions & 2 deletions src/components/TimetablePage/Timetable/Timetable.module.scss
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,8 @@
&--time {
& > div {
display: flex;
padding-left: 3px;
align-items: center;
justify-content: center;
font-family: Arial, Helvetica, sans-serif;
}
}
Expand Down Expand Up @@ -87,7 +87,7 @@
flex-direction: column;
position: absolute;
z-index: 3;
padding: 4px 2px 0;
padding: 6px 4px 0;
font-size: 0.9em;
line-height: 1.2;
font-family: NanumSquare, serif;
Expand Down
8 changes: 3 additions & 5 deletions src/components/TimetablePage/Timetable/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -83,9 +83,7 @@ function Timetable({
// eslint-disable-next-line react/no-array-index-key
key={`value-${index}`}
>
<div>
{value}
</div>
{value}
</div>
))}
<div style={{ height: `${rowHeight * 2}px` }}>
Expand All @@ -111,9 +109,9 @@ function Timetable({
key={lectureIndex}
style={{
backgroundColor: BACKGROUND_COLOR[lectureIndex % 11],
top: `${start * rowHeight}px`,
top: `${start * rowHeight + 1}px`,
width: isMobile ? undefined : `${columnWidth}px`,
height: `${(end - start + 1) * rowHeight}px`,
height: `${(end - start + 1) * rowHeight - 1}px`,
}}
>
<h4>
Expand Down

0 comments on commit 4c5ade1

Please sign in to comment.