Feat(#209, #240) 다시보기 페이지 프로그래스바 및 프롬프트 기능 구현 #241
Merged
Conversation
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
- 시간대에 맞춰 하이라이팅 - 프롬프트의 스크립트를 클릭시 해당 진행시간으로 이동
✅ Deploy Preview for boarlog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Jw705
approved these changes
Dec 10, 2023
Comment on lines
+15
to
+16
| // 추후 해당 다시보기의 전체 플레이 타임을 받아올 수 있어야 할 것 같습니다. | ||
| const TOTAL_MS_TIME_OF_REVIEW = 200000; |
There was a problem hiding this comment.
서버에 요청해서 받아오거나 음성 파일 길이로 계산해볼 수 있겠네요!
Comment on lines
+1
to
+3
| const MS_OF_SECOND = 1000; | ||
| const MS_OF_MINUTE = 60 * MS_OF_SECOND; | ||
| const MS_OF_HOUR = 60 * MS_OF_MINUTE; |
Comment on lines
+65
to
+70
| const startTime = Math.floor(+value.start / 1000) * 1000; | ||
|
|
||
| return startTime > progressMsTime; | ||
| }) - 1; | ||
| const lastStartTime = +scriptList[scriptList.length - 1]?.start; | ||
| if (Math.floor(lastStartTime / 1000) * 1000 <= progressMsTime) { |
There was a problem hiding this comment.
가능하면 아래 선언한 상수를 활용해서 이 부분도 매직넘버를 사용하지 않도록 고치면 좋을 것 같습니다.
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.
작업 개요
다시보기 페이지 프로그래스바 및 프롬프트 기능 구현
작업 사항
Feature: 다시보기 페이지 영상 프로그래스바 구현 close #209
Feature: 다시보기페이지 진행 시간에 따라 해당 시간대에 해당하는 프롬프트 하이라이팅 및 진행 시간 이동 기능 close #240
고민한 점들(필수 X)
설계
해당 설계의 이유: 시간이 없다고 생각해 이 컴포넌트를 누군가 이어받았을 때 최대한 쉽게 이용 가능하도록, 진행시간-stateProgressMsTime과 props로 주어지는 다시보기의 전체 길이(ms단위)-totalTime의 조작만으로 프로그래스 바와 prompt가 작동하도록 구현했습니다.
스크린샷(필수 X)
다시보기가 끝나면 버튼 비활성화 및 재생 정지
2023-12-10.3.00.08.mov
진행 시간에 맞춰 스크립트를 하이라이팅하고 클릭시 해당 진행시간으로 이동
2023-12-10.3.01.49.mov