Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI: Fix missing unassigned icon in System theme #11553

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shiina424
Copy link
Contributor

@shiina424 shiina424 commented Nov 22, 2024

Description

The unassigned icon is missing in the System theme.
This issue occurs in version 31.0-beta1 or later, but does not occur in version 30.2.3.
this issue was caused by this change. (maybe.) cb02696
I'm not sure if this change is correct, so please feel free to close it if it's not.

cb02696

Before

Before.mp4

After

After.mp4

Motivation and Context

I want the icons to be displayed correctly.
I’m not sure how many people are using the system theme, but I’ll fix it (even though I don’t use it myself).

How Has This Been Tested?

I checked with the local build.
I am not sure if the code change is correct, but the icon is now displaying.

Types of changes

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

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.

@RytoEX RytoEX added Bug Fix Non-breaking change which fixes an issue UI/UX Anything to do with changes or additions to UI/UX elements. labels Nov 22, 2024
@RytoEX RytoEX added this to the OBS Studio 31 milestone Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fix Non-breaking change which fixes an issue UI/UX Anything to do with changes or additions to UI/UX elements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants