Skip to content

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

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

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

Conversation

@ojongii
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Contributor

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

@skkuding-bot
Copy link
Copy Markdown

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
Copy Markdown
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
@sONg20NOW
Copy link
Copy Markdown
Contributor

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

@ojongii
Copy link
Copy Markdown
Contributor Author

ojongii commented Mar 19, 2026

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

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

@sONg20NOW
Copy link
Copy Markdown
Contributor

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

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

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

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot bot commented Mar 27, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: 2ae5c217e15b05d621e4d97fa9ea3efcd4a6725d
Health Status: Healthy

Open Preview | View in Argo CD

@ojongii ojongii force-pushed the t2631-make-generator-checker-validator-page branch from 6d1ed3b to 2ae5c21 Compare April 2, 2026 07:47
@skkuding-bot
Copy link
Copy Markdown

skkuding-bot bot commented Apr 2, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: 7bb1ca324c71d045b1bc009bd98056db89b6fc2b
Health Status: Healthy

Open Preview | View in Argo CD

@skkuding-bot
Copy link
Copy Markdown

skkuding-bot bot commented Apr 2, 2026

Syncing Preview App Succeeded

Application: frontend
Revision: 7b9d0b4924695773f27cae58020d89a3c9603e8b
Health Status: Healthy

Open Preview | View in Argo CD

@sONg20NOW
Copy link
Copy Markdown
Contributor

일단 너무 고생하셨습니다!! 몇가지만 고치면 될 것 같아요~

</div>

<div className="px-6 py-7">
{activeTab === 'generator' && (
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

요기서 특정 탭에서 파일 업로드를 하고 다른 탭으로 가면 업로드했던 그 파일이 사라지는 문제가 있어요! 현재 코드는 하나의 FileUploadSection 컴포넌트만 살려두게 되어있는데 세 개 모두 살려두되 선택되지 않은 탭 페이지에 대해서는 hidden으로 두던가 해야할 것 같아요~

text: '특수 채점',
subText: '특수 채점 기능',
Component: CheckerPage
label: 'Tools',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

SNB 수정하시고 추가적으로 스켈레톤 코드도 이에 맞게 수정해야 할 것 같아용!

</p>
<div className="bg-color-neutral-99 border-color-cool-neutral-90 flex flex-col gap-2 rounded-[8px] border p-4">
{FORMAT_EXAMPLES.map((example, index) => (
<div key={index} className="flex items-center gap-2">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

bullet이랑 텍스트 사이 너비가 피그마 상보다 좁아요! 이 페이지의 bullet 크기가 8x8이 아니어서 그런 것 같습니당!

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