Skip to content

fix: Removed videoView on cell-reuse - #2750

Open
SystemKeeper wants to merge 2 commits into
mainfrom
fix/noid/removed-video-view
Open

SystemKeeper wants to merge 2 commits into
mainfrom
fix/noid/removed-video-view

Conversation

@SystemKeeper

Copy link
Copy Markdown
Collaborator

Depending on layouting and cell-reuse it is possible that we remove the wrong video view, essentially leaving the participant cell without a video. Therefore we need to ensure that the view is still the correct one.
Additionally this PR moves the cell code to swift, so best to check both commits individually.

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

@SystemKeeper
SystemKeeper requested a review from Ivansss October 7, 2026 16:49
@SystemKeeper SystemKeeper changed the title Fix/noid/removed video view fix: Removed videoView on cell-reuse Oct 7, 2026
Assisted-by: ClaudeCode:claude-opus-5.5
Signed-off-by: Marcel M眉ller <marcel-mueller@gmx.de>
Assisted-by: ClaudeCode:claude-opus-5.5
Signed-off-by: Marcel M眉ller <marcel-mueller@gmx.de>
@SystemKeeper
SystemKeeper force-pushed the fix/noid/removed-video-view branch from b27d893 to e7fe090 Compare October 7, 2026 17:34
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.

1 participant