Skip to content

frontend: Use styling for source items visual appearance - #13833

Merged
RytoEX merged 2 commits into
obsproject:masterfrom
Warchamp7:hidden-sceneitem-menu-fix
Sep 1, 2026
Merged

frontend: Use styling for source items visual appearance#13833
RytoEX merged 2 commits into
obsproject:masterfrom
Warchamp7:hidden-sceneitem-menu-fix

Conversation

@Warchamp7

Copy link
Copy Markdown
Member

Description

The icon and text label for items in the sources list was previously enabled/disabled based on the scene item visibility state. As of Qt 6.11 Qt no longer triggers context menus when right clicking a disabled QLabel. This means right-clicking a hidden scene item on the source icon or name would not show a context menu.

Updates SourceTreeItem to use styling classes via idian helpers instead of changing the enabled state of these labels.

Motivation and Context

Fixes context menus not appearing for hidden scene items when clicking certain regions of the source list.

How Has This Been Tested?

Toggled scene item visibility and right-clicked on sources in the sources list.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Code cleanup (non-breaking change which makes code smaller or more readable)

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.

Updates idian utils to also support QLabel widgets for icon coloring.
The icon and text label for items in the source list was previously
enabled/disabled based on scene item visibility state. As of Qt 6.11
Qt no longer triggers context menus when right clicking a disabled
QLabel. This means right-clicking a hidden scene item would not
show a context menu.

Updates SourceTreeItem to use styling classes via idian helpers
instead of changing the enabled state of these labels.
@Warchamp7 Warchamp7 added this to the OBS Studio 33.0 milestone Aug 28, 2026
@Warchamp7 Warchamp7 added the area/ui-ux Anything to do with changes or additions to UI/UX elements. label Aug 28, 2026
@github-project-automation github-project-automation Bot moved this to Ready For Review in 33.0 Release Tracker Aug 28, 2026
@RytoEX
RytoEX requested review from PatTheMav and RytoEX August 29, 2026 03:34
@RytoEX RytoEX self-assigned this Aug 29, 2026
@RytoEX RytoEX added kind/bug Categorizes issue or PR as related to a bug. kind/cleanup Non-breaking change which makes code smaller or more readable labels Aug 29, 2026
@RytoEX
RytoEX merged commit e8c4e71 into obsproject:master Sep 1, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this from Ready For Review to Merged in 33.0 Release Tracker Sep 1, 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/bug Categorizes issue or PR as related to a bug. kind/cleanup Non-breaking change which makes code smaller or more readable

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants