Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
관리자 대회 생성(및 수정) 페이지의 UI/UX를 문제 생성 페이지와 유사한 스타일로 정리하고, 허용 IP 범위 입력 UX를 개선하는 PR입니다.
Changes:
- 대회 생성 폼 레이아웃을 카드형 UI로 개편(라디오/토글/입력 필드 구성 변경)
- 허용 IP 범위 입력에 “추가” 버튼을 별도 제공하고, 마지막 1개는 삭제 불가 처리
- 대회 수정 시 allowed_ip_ranges 데이터를 입력용 형태로 매핑 로직 간소화
Comment on lines
+68
to
+75
| <label class="custom-label">설정</label> | ||
| <div class="toggle-row"> | ||
| <div class="toggle-item"> | ||
| <span class="toggle-label"> | ||
| {{ $t('m.Contest_Status') }} | ||
| <el-tooltip content="활성화하면 참가자에게 대회가 노출됩니다." placement="top"> | ||
| <i class="el-icon-question help-icon"></i> | ||
| </el-tooltip> |
There was a problem hiding this comment.
설정 섹션 라벨과 3개 el-tooltip의 content가 한국어 문자열로 하드코딩되어 있어 다국어(i18n) 환경에서 번역이 적용되지 않습니다. 이 프로젝트의 다른 관리자 화면들은 문자열을 $t('m.*')로 처리하고 있으니(예: frontend/src/pages/admin/views/general/HomeBannerManagement.vue:32의 $t('m.Setting')) 동일하게 번역 키를 사용하도록 변경하는 편이 좋습니다.
Comment on lines
+71
to
+80
| <span class="toggle-label"> | ||
| {{ $t('m.Contest_Status') }} | ||
| <el-tooltip content="활성화하면 참가자에게 대회가 노출됩니다." placement="top"> | ||
| <i class="el-icon-question help-icon"></i> | ||
| </el-tooltip> | ||
| </span> | ||
| <label class="spj-toggle"> | ||
| <input type="checkbox" v-model="contest.visible" /> | ||
| <span class="spj-toggle-track" :class="{ 'is-on': contest.visible }"></span> | ||
| </label> |
There was a problem hiding this comment.
커스텀 토글에서 <input type="checkbox">가 display:none 처리되어 있고, 입력을 감싼 <label class="spj-toggle"> 안에 접근 가능한 텍스트(라벨명)가 없어 스크린리더가 체크박스의 이름을 읽지 못할 가능성이 큽니다. 현재 텍스트는 별도의 <span class="toggle-label">에 있어 입력과 연결되지 않으므로, 토글 텍스트를 같은 <label> 안에 포함시키거나 aria-label/aria-labelledby로 입력과 텍스트를 연결해 주세요.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changelog
Testing
2026-04-17.1.44.19.mov
Ops Impact
N/A
Version Compatibility
N/A