Conversation
- `TextFieldShort`, `SearchableSongList` 등 주요 컴포저블에서 하드코딩된 "Text Field" 문자열을 `stringResource`를 사용하도록 수정 - `CaptionLabeledTextField`에 `placeholder` 파라미터를 추가하여 외부에서 플레이스홀더를 주입할 수 있도록 개선 - `afternote_editor_message_text_field_placeholder` 리소스 문자열을 "입력해 주세요"로 구체화 - 텍스트 및 레이아웃 변경에 따른 core-ui 모듈의 스크린샷 테스트 레퍼런스 이미지 업데이트
|
❌ Screenshot test fail — baseline 과 CI rendered PNG 차이. 📦 Artifact: https://github.com/Afternote/Afternote-FE/actions/runs/26936796018 (페이지 하단 Artifacts → 🔍 확인 위치 (artifact zip 내부):
📝 조치 가이드:
CI ↔ 로컬 환경 차이는 본 워크플로의 container 단계가 root fix 한다 (README 의 "Compose Preview Screenshot Testing (docker baseline)" 섹션 참조). |
- `AfternotePopupCardLayout` 스크린샷 테스트 레퍼런스 이미지 업데이트 - `BottomBar` 스크린샷 테스트 레퍼런스 이미지 업데이트 - `ProfileImagePicker` 플레이스홀더 스크린샷 테스트 레퍼런스 이미지 업데이트
- `CustomServiceDialog` 스크린샷 테스트의 레퍼런스 이미지 최신화 - `MemorialPlaylistScreen` 스크린샷 테스트의 레퍼런스 이미지 최신화
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
1 participant
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.
📌𝘐𝘴𝘴𝘶𝘦𝘴
Closes #394 — fix(core-ui): TextField 기본 placeholder 더미 노출 수정
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
AfternoteTextField의 placeholder 기본 파라미터를 하드코딩"Text Field"→ string 리소스core_ui_text_field_placeholder("입력해 주세요")로 교체 (TextFieldShort.kt)CaptionLabeledTextField에placeholder파라미터 추가 + 하위 전파SearchableSongList노래 검색 필드: 하드코딩"Text Field"→ 기존song_search_placeholderstring 사용📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵
placeholder 텍스트 변경 — screenshot baseline 4개 갱신으로 반영 (Compose Preview / screenshot test 검증). 별도 캡처 없음.
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴
feature/timeletter·feature/setting의 동일"Text Field"하드코딩은 타 담당자 영역이라 이 PR 범위에서 제외했습니다../gradlew :core:ui:compileDebugKotlin :feature:afternote:presentation:compileDebugKotlin→ BUILD SUCCESSFUL