Skip to content

frontend: Don't reload scene collection after rename#13617

Open
Dankirk wants to merge 1 commit into
obsproject:masterfrom
Dankirk:scene-collection-rename
Open

frontend: Don't reload scene collection after rename#13617
Dankirk wants to merge 1 commit into
obsproject:masterfrom
Dankirk:scene-collection-rename

Conversation

@Dankirk

@Dankirk Dankirk commented Jul 2, 2026

Copy link
Copy Markdown

Description

Skip unnecessary scene collection reload during rename for better ux.

Motivation and Context

The goal is better user experience. For example the reload after renaming caused active audio/video files to reset to beginning.

As a side effect this also works around a hang issue described in #13495, since the reload is skipped. The underlying issue should still be addressed, but is outside of this pr scope.

How Has This Been Tested?

After renaming I monitored the scene collection is handled as should while switching scenes, collections, interacting with sources and restarting app.

Types of changes

  • Tweak (non-breaking change to improve existing functionality)

Checklist:

  • I have read the contributing document.
  • My code has been run through clang-format.
  • My code follows the project's style guidelines
  • My code is not on the master branch.
  • My code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

Skip unnecessary scene collection reload during rename for better ux.
@RytoEX RytoEX requested review from PatTheMav, RytoEX and Warchamp7 July 2, 2026 19:57
@WizardCM WizardCM added the kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. label Jul 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement Enhancements are not bugs or new features but can improve usability or performance.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants