Skip to content

feat(fe): make generator, checker, validator page#3491

Open
ojongii wants to merge 4 commits intomainfrom
t2631-make-generator-checker-validator-page
Open

feat(fe): make generator, checker, validator page#3491
ojongii wants to merge 4 commits intomainfrom
t2631-make-generator-checker-validator-page

Conversation

@ojongii
Copy link
Contributor

@ojongii ojongii commented Mar 17, 2026

Description

문제 생성 탭 중에서 generator, checker, validator 페이지를 만들었습니다.
공통된 코드를 컴포넌트화하고 디자인, 파일 업로드 기능을 구현했어요.

image

Additional context

close TAS-2631


Before submitting the PR, please make sure you do the following

@ojongii ojongii requested review from sONg20NOW and woo943 March 17, 2026 13:49
@ojongii ojongii self-assigned this Mar 17, 2026
@ojongii ojongii added ⛳️ team-frontend 🍊squad-유자차 스쿼드 유자차 작업물입니다 labels Mar 17, 2026
@ojongii ojongii added this to Codedang Mar 17, 2026
@github-project-automation github-project-automation bot moved this to Pending ✋ in Codedang Mar 17, 2026
@sONg20NOW sONg20NOW added the preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 label Mar 17, 2026
@skkuding-bot
Copy link

skkuding-bot bot commented Mar 17, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: dec47f87c78fef5d01fe7ff25d9c18e6ad05b8da
Health Status: Healthy

Open Preview | View in Argo CD

@sONg20NOW
Copy link
Contributor

컴포넌트화하니 깔끔하네요!! 👍 아래 자잘한 수정사항만 확인해주세요~

@skkuding-bot
Copy link

skkuding-bot bot commented Mar 18, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: bf7607afee59ac2a18b83cd4b82d2271ffec80b1
Health Status: Healthy

Open Preview | View in Argo CD

Copy link
Contributor

@woo943 woo943 left a comment

Choose a reason for hiding this comment

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

LGTM~ 고생하셨어요!

@github-project-automation github-project-automation bot moved this from Pending ✋ to Approved 👌 in Codedang Mar 18, 2026
>
<div className="flex gap-4">
<span className="bg-color-neutral-99 flex h-12 w-12 items-center justify-center rounded-[6.4px] border">
<Image src={fileIcon} alt="file" width={20} height={20} />
Copy link
Contributor

Choose a reason for hiding this comment

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

아이콘 크기 24x24로 바꿔주세요~

)}
</div>

<p className="text-sub1_sb_18 mb-2">포맷 예시</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

텍스트 색상 피그마대로 cool-neutral-30으로 변경해야 할 것 같아요~

{files.length === 0 ? (
<div className="bg-color-neutral-99 flex flex-col items-center rounded-[12px] py-20 text-center">
<Image
src={infoIcon}
Copy link
Contributor

Choose a reason for hiding this comment

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

피그마랑 살짝 색상이 달라서 새로운 svg 파일을 만들어야 할 것 같은데 제 PR에 이미 만들어놔서 제 거 머지되면 그거 사용해주시면 될 것 같아요!

(이름은 info-icon-gray.svg 입니다!)

@sONg20NOW
Copy link
Contributor

추가적으로 저는 솔루션 파일 업로드 할 때 기존 파일 위에 추가되도록 했는데 종희님처럼 아래에 추가하는 편이 나으려나요?
전 파일 추가하고 최근 파일 확인하려면 스크롤 내려야 하는 게 번거로울 것 같아서 그랬는데, 이 페이지의 경우에는 한 파일당 차지하는 크기가 그렇게 크지 않아서 아래에 추가하는 게 맞는 것 같기도 하고...
아무래도 통일성 있게 가는 게 UX 측면에서 맞는 것 같은데 종희님 생각은 어떠신가요?

@ojongii
Copy link
Contributor Author

ojongii commented Mar 19, 2026

추가적으로 저는 솔루션 파일 업로드 할 때 기존 파일 위에 추가되도록 했는데 종희님처럼 아래에 추가하는 편이 나으려나요? 전 파일 추가하고 최근 파일 확인하려면 스크롤 내려야 하는 게 번거로울 것 같아서 그랬는데, 이 페이지의 경우에는 한 파일당 차지하는 크기가 그렇게 크지 않아서 아래에 추가하는 게 맞는 것 같기도 하고... 아무래도 통일성 있게 가는 게 UX 측면에서 맞는 것 같은데 종희님 생각은 어떠신가요?

solution 탭 PR에 남긴 댓글 확인 부탁드립니다!

@sONg20NOW
Copy link
Contributor

추가적으로 저는 솔루션 파일 업로드 할 때 기존 파일 위에 추가되도록 했는데 종희님처럼 아래에 추가하는 편이 나으려나요? 전 파일 추가하고 최근 파일 확인하려면 스크롤 내려야 하는 게 번거로울 것 같아서 그랬는데, 이 페이지의 경우에는 한 파일당 차지하는 크기가 그렇게 크지 않아서 아래에 추가하는 게 맞는 것 같기도 하고... 아무래도 통일성 있게 가는 게 UX 측면에서 맞는 것 같은데 종희님 생각은 어떠신가요?

solution 탭 PR에 남긴 댓글 확인 부탁드립니다!

넵 그럼 종희님 방식으로 저도 작업하도록 하겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview 이 라벨이 붙어있어야 프론트엔드 Preview 환경이 생성됩니다 🍊squad-유자차 스쿼드 유자차 작업물입니다 ⛳️ team-frontend

Projects

Status: Approved 👌

Development

Successfully merging this pull request may close these issues.

3 participants