Skip to content

gui: ensure QT5 and QT6 behave the same#10613

Merged
maliberty merged 1 commit into
The-OpenROAD-Project:masterfrom
gadfort:gui-fix-regex
Jun 8, 2026
Merged

gui: ensure QT5 and QT6 behave the same#10613
maliberty merged 1 commit into
The-OpenROAD-Project:masterfrom
gadfort:gui-fix-regex

Conversation

@gadfort

@gadfort gadfort commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix behavior of gui::set_display_controls

Type of Change

  • Bug fix

Impact

In Qt5 gui::set_display_controls "*" visible false works and sets all elements to not visible
in Qt6 gui::set_display_controls "*" visible false matched nothing, this fixes it to match.

Verification

  • I have verified that the local build succeeds (./etc/Build.sh).
  • I have run the relevant tests and they pass.
  • My code follows the repository's formatting guidelines.
  • I have signed my commits (DCO).

Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
@gadfort gadfort requested a review from a team as a code owner June 8, 2026 14:51
@gadfort gadfort requested a review from maliberty June 8, 2026 14:51
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

@maliberty maliberty enabled auto-merge June 8, 2026 14:55
@maliberty maliberty merged commit f2f20fa into The-OpenROAD-Project:master Jun 8, 2026
17 checks passed
@gadfort gadfort deleted the gui-fix-regex branch June 8, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants