Skip to content

frontend: Demangle ScreenshotObj and OBSBasic - #13037

Open
Warchamp7 wants to merge 2 commits into
obsproject:masterfrom
Warchamp7:demangle-screenshot2
Open

frontend: Demangle ScreenshotObj and OBSBasic#13037
Warchamp7 wants to merge 2 commits into
obsproject:masterfrom
Warchamp7:demangle-screenshot2

Conversation

@Warchamp7

Copy link
Copy Markdown
Member

Description

This adds a signal to ScreenshotObj so it doesn't have to directly call OBSBasic. Updated after the changes due to the Add Source dialog.

Revised version that supersedes #12186

Motivation and Context

Less friends for OBSBasic and better code.

How Has This Been Tested?

Took screenshots to make sure they still worked.

Types of changes

  • Code cleanup (non-breaking change which makes code smaller or more readable)

Checklist:

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

@Warchamp7 Warchamp7 added kind/cleanup Non-breaking change which makes code smaller or more readable area/ui-ux Anything to do with changes or additions to UI/UX elements. labels Jan 16, 2026
Comment thread frontend/widgets/OBSBasic.hpp Outdated
Comment thread frontend/widgets/OBSBasic_Screenshots.cpp
@Warchamp7 Warchamp7 added this to the OBS Studio 32.2 milestone May 6, 2026
@RytoEX

RytoEX commented May 7, 2026

Copy link
Copy Markdown
Member

This currently has merge conflicts.

@Warchamp7
Warchamp7 force-pushed the demangle-screenshot2 branch 2 times, most recently from 0b2cfe8 to c2a787e Compare May 8, 2026 02:05
@Warchamp7

Copy link
Copy Markdown
Member Author

This currently has merge conflicts.

Fixed.

@Warchamp7
Warchamp7 force-pushed the demangle-screenshot2 branch from c2a787e to 548c3d1 Compare May 8, 2026 02:16
@Warchamp7 Warchamp7 removed this from the OBS Studio 32.2 milestone Jul 30, 2026
@github-project-automation github-project-automation Bot moved this to Ready For Review in 33.0 Release Tracker Aug 15, 2026
Co-Authored-By: Clayton Groeneveld <19962531+cg2121@users.noreply.github.com>
@Warchamp7
Warchamp7 force-pushed the demangle-screenshot2 branch from 548c3d1 to d27256f Compare August 17, 2026 21:16
Comment thread frontend/utility/ScreenshotObj.cpp Outdated
@github-project-automation github-project-automation Bot moved this from Ready For Review to Requires Changes in 33.0 Release Tracker Aug 18, 2026
@Warchamp7
Warchamp7 force-pushed the demangle-screenshot2 branch from d27256f to 8ceadc7 Compare August 18, 2026 20:20
@Warchamp7 Warchamp7 moved this from Requires Changes to Ready For Review in 33.0 Release Tracker Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ui-ux Anything to do with changes or additions to UI/UX elements. kind/cleanup Non-breaking change which makes code smaller or more readable

Projects

Status: Ready For Review

Development

Successfully merging this pull request may close these issues.

3 participants