Skip to content

frontend: Refactor Idian widgets - #13765

Open
Warchamp7 wants to merge 1 commit into
obsproject:masterfrom
Warchamp7:idian-refactor
Open

frontend: Refactor Idian widgets#13765
Warchamp7 wants to merge 1 commit into
obsproject:masterfrom
Warchamp7:idian-refactor

Conversation

@Warchamp7

Copy link
Copy Markdown
Member

Description

Refactors the Idian widgets based after some slight usage in other PRs.

This has been extracted out from #13433

Motivation and Context

The idian widgets were built when we had a much lower understanding of how Qt expects you to build custom complex widgets and without a direct use-case.

They were also built far too rigid for what every attempted use of them thus far has necessitated. These changes break them into simpler 'pieces' and provide getters for internals that are intended to be accessed.

How Has This Been Tested?

Included in #13433

Types of changes

  • Tweak (non-breaking change to improve existing functionality)
  • 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.

The idian widgets were built when we had a much lower
understanding of how Qt expects you to build custom
complex widgets and without a direct use-case.

They were also built far too rigid for what every attempted
use of them thus far has necessitated. These changes break
them into simpler 'pieces' and provide getters for internals
that are intended to be accessed.
@Warchamp7 Warchamp7 added the area/ui-ux Anything to do with changes or additions to UI/UX elements. label Aug 7, 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant