Skip to content
This repository was archived by the owner on Feb 10, 2026. It is now read-only.

FCE 2148: Picture in picture improvements on iOS - #507

Merged
anna1901 merged 8 commits into
mainfrom
fce-2148/pip-improvements-ios
Oct 28, 2025
Merged

FCE 2148: Picture in picture improvements on iOS#507
anna1901 merged 8 commits into
mainfrom
fce-2148/pip-improvements-ios

Conversation

@anna1901

@anna1901 anna1901 commented Oct 27, 2025

Copy link
Copy Markdown
Contributor

Description

  • Rewrite the split screen shown in pip as a swiftUI view
  • fix showing active speaker
  • bonus: add swift formatting rule to editorconfig

TODO: There are still some problems with VAD activation and metadata detection which will be fixed in FCE-2189

Documentation impact

  • Documentation update required
  • Documentation updated in another PR
  • No documentation update required

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)

@linear

linear Bot commented Oct 27, 2025

Copy link
Copy Markdown

@MiloszFilimowski MiloszFilimowski left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tested it and works (and looks) great!

Comment thread .editorconfig

[*.swift]
indent_style = space
indent_size = 4 No newline at end of file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

praise: 💯

}

func updateUIView(_ uiView: UIView, context: Context) {
// Update if needed

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

question: ?

@MiloszFilimowski MiloszFilimowski left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tested it and works (and looks) great!

@anna1901
anna1901 merged commit b71dae9 into main Oct 28, 2025
4 checks passed
@anna1901
anna1901 deleted the fce-2148/pip-improvements-ios branch October 28, 2025 18:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants