Skip to content

Slider improvements and a few bug fixes - #150

Open
mtayloooor wants to merge 5 commits into
itaybre:masterfrom
mtayloooor:slider-improvements
Open

mtayloooor wants to merge 5 commits into
itaybre:masterfrom
mtayloooor:slider-improvements

Conversation

@mtayloooor

@mtayloooor mtayloooor commented Oct 31, 2026 •

Copy link
Copy Markdown

Slider improvements

  • Move a slider while auto is on. Exposure, White Balance, Focus and Hue sliders are no longer disabled in auto mode. Moving one switches that control to manual (the toggle and Auto badge update) and applies the value.
  • Double-click to reset. Double-clicking a slider's thumb resets it to the default value the camera reports (defaultValue), and applies it. Works for every slider, including Pan/Tilt and Gain.

Slider takes an optional defaultValue; GenericControl passes it through and handles the auto-to-manual switch. Exposure does the same in ExposureView.

Bug fixes (one commit each)

  • Pan/Tilt overflow: UVCMultipleIntControl could overflow when combining the two values; now uses wrapping arithmetic.
  • Config descriptor lifetime: the descriptor from GetConfigurationDescriptorPtr was parsed after the device interface was released, reading freed memory on newer macOS. It's now parsed while the interface is alive.
  • Move to Applications: LetsMove built paths with URL(string:) / absoluteString, giving file:// paths that FileManager rejects. Now uses URL(fileURLWithPath:) / .path.
  • Camera preview: AVCaptureSession start/stop now runs on a serial queue instead of the main thread, so the UI doesn't freeze while a camera starts or switches. The preview layer is added once instead of on every layout().

Tested with a UVC camera as part of a larger build on my fork; this branch on its own has been typechecked but not built in Xcode.

🤖 Generated with Claude Code

MTAY and others added 5 commits October 31, 2026 13:27
- Exposure, White Balance, Focus and Hue sliders can be moved while
  auto is on; moving one switches that control to manual
- Double-clicking a slider's thumb resets it to the camera's default

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mtayloooor mtayloooor changed the title Let auto sliders move and double-click to reset Slider improvements and a few bug fixes Oct 31, 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.

1 participant