Skip to content

Panorama video viewer (copied from viewer) - #57

Open
kyle-fitzp wants to merge 4 commits into
masterfrom
360-video
Open

Panorama video viewer (copied from viewer)#57
kyle-fitzp wants to merge 4 commits into
masterfrom
360-video

Conversation

@kyle-fitzp

Copy link
Copy Markdown

Same changes from OSH-Viewer: Botts-Innovative-Research/osh-viewer#284

@salsajeries salsajeries left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Adding @Cardy2 as reviewer

@salsajeries
salsajeries requested a review from Cardy2 July 1, 2026 17:17
Comment thread source/core/ui/view/video/VideoView.js Outdated
Comment thread source/core/ui/view/video/YUV360Canvas.js
Comment thread source/core/ui/view/video/YUV360Canvas.js
Comment thread source/core/ui/view/video/FFMPEG360View.js
Comment thread source/core/ui/view/video/VideoView.js Outdated

@Cardy2 Cardy2 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good. Mostly small fixes and need to add some cleanup detaching the event listeners in 360Canvas

@Cardy2 Cardy2 self-assigned this Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants