Skip to content

feat: migrate LogPanel to QTable model/view#958

Open
d-perl wants to merge 4 commits intomainfrom
feat/table-log-panel
Open

feat: migrate LogPanel to QTable model/view#958
d-perl wants to merge 4 commits intomainfrom
feat/table-log-panel

Conversation

@d-perl
Copy link
Contributor

@d-perl d-perl commented Nov 27, 2025

Migrates the LogPanel to a table model and view, adds tests for core functionality.

@d-perl d-perl force-pushed the feat/table-log-panel branch 12 times, most recently from 6795513 to eb1b1d8 Compare November 28, 2025 13:42
@codecov
Copy link

codecov bot commented Nov 28, 2025

Codecov Report

❌ Patch coverage is 86.36364% with 39 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
bec_widgets/widgets/utility/logpanel/logpanel.py 86.02% 28 Missing and 11 partials ⚠️

📢 Thoughts on this report? Let us know!

@d-perl d-perl force-pushed the feat/table-log-panel branch 3 times, most recently from ab85263 to 4612ca9 Compare November 28, 2025 14:19
@d-perl d-perl marked this pull request as ready for review November 28, 2025 14:19
@d-perl d-perl requested review from cappel89, wakonig and wyzula-jan and removed request for wakonig and wyzula-jan November 28, 2025 14:20
@d-perl d-perl force-pushed the feat/table-log-panel branch 4 times, most recently from b0986f1 to fdb7008 Compare November 28, 2025 15:05
@wyzula-jan
Copy link
Contributor

wyzula-jan commented Nov 28, 2025

The performance is still throttled a lot after you add the LogPanel widget (hard to see on video but for example when you just try to drag stuff around it is much less responsive with the LogPanel):

Screen.Recording.2025-11-28.at.17.33.53.mov

@d-perl d-perl force-pushed the feat/table-log-panel branch from 2d3738f to a6107a2 Compare December 4, 2025 07:56
@d-perl d-perl requested a review from wyzula-jan December 4, 2025 08:10
@d-perl d-perl force-pushed the feat/table-log-panel branch 2 times, most recently from 2761bb0 to 8cebdad Compare December 15, 2025 10:17
@wyzula-jan wyzula-jan force-pushed the pre_release_v3 branch 7 times, most recently from c28fbf6 to b4ad75a Compare December 19, 2025 13:12
@d-perl d-perl force-pushed the feat/table-log-panel branch from 8cebdad to d7ee641 Compare March 6, 2026 09:17
@d-perl d-perl changed the base branch from pre_release_v3 to main March 6, 2026 09:17
@d-perl d-perl force-pushed the feat/table-log-panel branch 2 times, most recently from 098f829 to 9b806a2 Compare March 6, 2026 12:11
@d-perl d-perl force-pushed the feat/table-log-panel branch from 9b806a2 to 8d1539f Compare March 11, 2026 14:07
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.

2 participants