Skip to content

feat(dialog_sdk): add setDisabledRows support for QTableWidget#35

Merged
pabloinigoblasco merged 1 commit into
developmentfrom
feat/sdk-disabled-rows
Apr 6, 2026
Merged

feat(dialog_sdk): add setDisabledRows support for QTableWidget#35
pabloinigoblasco merged 1 commit into
developmentfrom
feat/sdk-disabled-rows

Conversation

@pabloinigoblasco
Copy link
Copy Markdown
Collaborator

@pabloinigoblasco pabloinigoblasco commented Apr 6, 2026

Summary

Adds support for disabling individual rows in QTableWidget via the dialog protocol.

  • WidgetData::setDisabledRows(name, rows) - Plugin-side builder method to mark rows as disabled
  • WidgetDataView::disabledRows(name) - Host-side accessor to read disabled rows
  • widget_binding.cpp - Applies Qt ItemIsEnabled and ItemIsSelectable flags per row

This enables plugins to grey out rows that should not be selectable, such as MCAP channels with message_count=0.

Test plan

  • Build passes on all platforms
  • Unit tests pass
  • Manual test with MCAP plugin (companion PR in pj-official-plugins)

Related

  • Companion PR in pj-official-plugins: feat(mcap): grey out channels with message_count=0

Adds support for disabling individual rows in QTableWidget via the
dialog protocol. Extends WidgetData with setDisabledRows(), WidgetDataView
with disabledRows(), and widget_binding.cpp to apply the Qt ItemIsEnabled
and ItemIsSelectable flags per row.

This enables plugins to grey out rows that shouldn't be selectable,
such as MCAP channels with message_count=0.

Contents:
- WidgetData::setDisabledRows(name, rows) builder method
- WidgetDataView::disabledRows(name) accessor for host
- widget_binding.cpp applies flags to QTableWidgetItems
@pabloinigoblasco pabloinigoblasco merged commit 4061375 into development Apr 6, 2026
2 checks passed
@pabloinigoblasco pabloinigoblasco deleted the feat/sdk-disabled-rows branch May 4, 2026 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant