발표분석 QA 2차 반영 - #176
Merged
Merged
Conversation
- `PresentationScheduleScreen`의 발표 이름 입력 필드(`PrezelTextField`)에 `status` 속성을 활용한 도움말 메시지 추가 - 발표 이름 입력 제약 사항(2~10자)을 안내하는 `feature_analysis_impl_presentation_name_helper` 문자열 리소스 추가
- `ReportNavKey`에 상단 바 표시 여부를 결정하는 `showTopAppBar` 필드 추가 - `AnalysisReportScreen` 및 `ReportScreenLayout`에서 `showTopAppBar` 값에 따라 상단 바를 선택적으로 렌더링하도록 수정 - 분석 완료 후 리포트 진입 시(`AnalysisEntryBuilder`) 상단 바를 숨기도록 설정 - 히스토리에서 리포트 진입 시(`HistoryEntryBuilder`) 상단 바가 표시되도록 설정 - `AnalysisReportScreen` 관련 Compose 프리뷰에 변경된 파라미터 반영
- `SituationAccordionType`에서 다음 섹션을 결정하는 `next` 속성 제거 - 카테고리, 목적, 스타일, 청중 선택 시 다음 섹션을 자동으로 확장하던 `onExpandedTypeChange` 호출 로직 삭제 - `nextExpandedType` 확장 함수 제거 및 관련 선택 이벤트 핸들러 단순화
- `VoiceRecordingContent`의 패딩 및 간격 조정 (V16 -> V4) - 스크립트 헤더 영역의 높이를 48.dp로 확장 및 아이콘 색상을 `iconDisabled`로 변경 - `PrezelButtonArea`에 배경색 노출 여부와 별개로 구분선을 제어할 수 있는 `showDivider` 파라미터 추가 - `VoiceRecordingButtonArea`에 배경색은 표시하되 구분선은 숨기도록 설정 적용 - `VoiceRecordingContent`의 다양한 상태(Idle, Recording, ReadyToPlay 등)에 대한 Compose 프리뷰 추가
- `PrezelVoiceChromeWave`에서 불필요한 애니메이션 상태(`activationProgress`, `volumeProgress`) 제거 - 최신 볼륨 데이터를 우선적으로 표시하도록 `sampleVolume` 로직 수정 - `VoiceRecordingChromeUi` 내 불필요한 `hideWaveform` 옵션 및 관련 조건부 로직 삭제 - `MIN_REACTIVE_VOLUME` 등 상수를 도입하여 파동 반응 임계값 및 높이 계산 로직 정교화
- `PresentationWordDetailResponse`의 `guideScript` 필드 타입을 `String?`로 변경하여 null 허용 - `PresentationMapper`에서 `guideScript` 매핑 시 `orEmpty()`를 사용하여 안전하게 처리하도록 수정
moondev03
approved these changes
Jul 26, 2026
This reverts commit 4dbbd43.
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
2 participants
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.
📌 작업 내용
2자부터 10자 이내로 입력해주세요안내 문구를 추가했습니다.PrezelButtonArea에서 배경과 divider를 독립적으로 제어할 수 있도록showDivider옵션을 추가했습니다.🧩 관련 이슈
closes #175