Skip to content

frontend: Fix deprecated header order in Add Source dialog#13666

Open
jcm93 wants to merge 1 commit into
obsproject:masterfrom
jcm93:fix/dialog-deprecated-header
Open

frontend: Fix deprecated header order in Add Source dialog#13666
jcm93 wants to merge 1 commit into
obsproject:masterfrom
jcm93:fix/dialog-deprecated-header

Conversation

@jcm93

@jcm93 jcm93 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes the placement of the "deprecated" header in the Add Source dialog such that all deprecated sources actually lie beneath it.

Motivation and Context

Some deprecated sources would lie above the "deprecated" header because the iterator that moved the sources removed items from the list while iterating through it (skipping some items).

This PR just makes the iterator operate in reverse order to avoid skipping any items as they're removed.

How Has This Been Tested?

Tested on macOS to verify that deprecated sources all now sit beneath the header. Verified that a deprecated source at the first or last position in the list is scooped up by the iterator.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

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.

@jcm93
jcm93 force-pushed the fix/dialog-deprecated-header branch from 83110da to 2abb2c6 Compare July 17, 2026 23:08
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.

2 participants