https://bugs.kde.org/show_bug.cgi?id=440855

            Bug ID: 440855
           Summary: BambooTracker instrument graphs invisible on Breeze
                    master branch
           Product: Breeze
           Version: unspecified
          Platform: Other
                OS: Linux
            Status: REPORTED
          Severity: normal
          Priority: NOR
         Component: QStyle
          Assignee: plasma-b...@kde.org
          Reporter: nyanpas...@tuta.io
                CC: noaha...@gmail.com
  Target Milestone: ---

SUMMARY
With Breeze theme's recent master installed, BambooTracker's instrument
editor's graphs are replaced with white rectangles.

STEPS TO REPRODUCE
1. Install Breeze at or after f6ef4f4738a5d3b6a07b7b11451af19635c8acd9 "Draw
background for frames".
2. Install and run BambooTracker.
3. Create and double-click an instrument.

OBSERVED RESULT
The rectangles on the bottom right have line graphs.

EXPECTED RESULT
The rectangles on the bottom right are empty and white.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.22.4
KDE Frameworks Version: 5.84.0
Qt Version: 5.15.2
Kernel Version: 5.13.8-zen1-1-zen (64-bit)
Graphics Platform: X11
Processors: 12 × AMD Ryzen 5 5600X 6-Core Processor
Memory: 15.6 GiB of RAM
Graphics Processor: NVIDIA GeForce GT 730/PCIe/SSE2

ADDITIONAL INFORMATION
I looked at the GUI in GammaRay. Each row ("Operator N") of the instrument
dialog is a FMOperatorTable object. The instrument graph (envFrame) is a
QFrame, not a subclass with a custom paint operation.

https://github.com/BambooTracker/BambooTracker/blob/master/BambooTracker/gui/instrument_editor/fm_operator_table.cpp#L205-L216

FMOperatorTable::eventFilter() intercepts QEvent::Paint and paints a graph onto
the frame (copied from QPixmap envmap_). On newer versions of Breeze, according
to GammaRay's "Paint Analyzer", it then gets overwritten by Breeze QFrame's
standard paint event.

BambooTracker's method of painting the graph is unusual. But is it wrong?
What's a better way of doing it? Should Breeze change to make this work?

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to