Skip to content

Conversation

@seongminn
Copy link
Member

🌍 이슈 번호

✅ 작업 내용

  • 대회 영상 컴포넌트를 추가합니다.

image

📝 참고 자료

♾️ 기타

  • 탭을 새로 클릭할 때마다 iframe 요소를 새롭게 렌더링합니다. 그래서 영상이 로드되는 데에 시간이 걸리는 것처럼 보입니다. 만약 라이브러리를 사용하여 개선할 수 있다면 추후 생각해봐도 좋을 것 같습니다!

@seongminn seongminn added the feature New features label Mar 22, 2024
@seongminn seongminn self-assigned this Mar 22, 2024
@netlify
Copy link

netlify bot commented Mar 22, 2024

Deploy Preview for hufscheer ready!

Name Link
🔨 Latest commit fb56a5e
🔍 Latest deploy log https://app.netlify.com/sites/hufscheer/deploys/65fedbd48b04d000096e8423
😎 Deploy Preview https://deploy-preview-132--hufscheer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 53
Accessibility: 71
Best Practices: 83
SEO: 55
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Mar 22, 2024

Deploy Preview for hufscheer-manager ready!

Name Link
🔨 Latest commit fb56a5e
🔍 Latest deploy log https://app.netlify.com/sites/hufscheer-manager/deploys/65fedbd4bc07ff00086d3268
😎 Deploy Preview https://deploy-preview-132--hufscheer-manager.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@ohprettyhak ohprettyhak left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

borderRadius: theme.spaces.xs,
border: 'none',

aspectRatio: '16 / 9',
Copy link
Contributor

Choose a reason for hiding this comment

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

Video 영역은 항상 16:9 비율 고정일까요? 세로 모드나 다른 비율 관련해서는 생각을 해보아야 겠네요.

Copy link
Member Author

Choose a reason for hiding this comment

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

이 부분은 유튜브에서 권장하는 표준 가로, 세로 비율이 16:9라서 이렇게 해두었습니다! 관련해서는 추후 팀원들이랑 다같이 논의해봐도 좋을 것 같습니다ㅎㅎ

Copy link
Contributor

@HiimKwak HiimKwak left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 저는 대회 영상을 highlight로 표현해서 경기목록 컴포넌트에서 highlight로 tab state를 관리했는데, video라는 단어랑 혼용되면 헷갈릴 것 같아 통일하는게 좋을 것 같아요. 제 생각에 사용자한테 노출되는 단어는 video보단 highlight가 더 와닿을거라 생각하는데 어떠신가요?

@seongminn
Copy link
Member Author

용어를 통일하자는 데에는 완전 찬성입니다!

제 의견은, 먼저 경기 영상이 하이라이트 영상만을 다루는 것이 아니라 중계 영상을 포함할 수도 있다는 점에서 좀 더 범용적인 단어를 사용하는 게 어떨까 했어요. 그리고 Video나, Highlight와 같은 용어가 사용자에게 노출될 일은 크게 없을 것이고, 팀 내부에서 관리하기 좋은 단어를 사용하면 좋을 것 같다는 생각이 들어요!

@seongminn
Copy link
Member Author

일단 기존에 작업된 내용인 Highlight로 통일해서 수정해두겠습니다!

@seongminn seongminn merged commit f549a23 into main Mar 23, 2024
@seongminn seongminn deleted the feat/game-video branch March 23, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 대회 영상 추가

4 participants