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

            Bug ID: 487061
           Summary: Bar chart width not enough for System Monitor on
                    horizontal panel with multiple bars
    Classification: Applications
           Product: ksysguard
           Version: 6.0.4
          Platform: Ubuntu
                OS: Linux
            Status: REPORTED
          Severity: normal
          Priority: NOR
         Component: general
          Assignee: ksysguard-b...@kde.org
          Reporter: fxzj...@163.com
                CC: plasma-b...@kde.org
  Target Milestone: ---

Created attachment 169506
  --> https://bugs.kde.org/attachment.cgi?id=169506&action=edit
Comparison of the bar chart

SUMMARY
After "Fix linecharts and bar charts for small sizes"
(https://invent.kde.org/plasma/libksysguard/-/merge_requests/339), 
width of bar chart is too small for bar charts with multiple bars like
"Individual Core Usage" to be shown.

STEPS TO REPRODUCE
1. add a horizontal panel to desktop
2. add a Individual Core Usage to the panel

OBSERVED RESULT
Bars cannot be shown (actually too narrow to show; if there's only 2 bars it
can be shown)

EXPECTED RESULT
Usage of all cores are shown

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Ubuntu 22.04
(available in About System)
KDE Plasma Version: 6.0.4
KDE Frameworks Version: 6.3.0
Qt Version: 6.6.3

ADDITIONAL INFORMATION
For this scenario, I'd suggest revert the minimal width back:
```
diff --git a/faces/facepackages/barchart/contents/ui/CompactRepresentation.qml
b/faces/facepackages/barchart/contents/ui/CompactRepresentation.qml
index b7740c43..c847491f 100644
--- a/faces/facepackages/barchart/contents/ui/CompactRepresentation.qml
+++ b/faces/facepackages/barchart/contents/ui/CompactRepresentation.qml
@@ -21,7 +21,7 @@ import org.kde.quickcharts.controls as ChartControls

 Faces.SensorFace {
     id: root
-    Layout.minimumWidth: Kirigami.Units.gridUnit
+    Layout.minimumWidth: formFactor == Faces.SensorFace.Horizontal ?
Kirigami.Units.gridUnit / 2 * barChart.barCount : Kirigami.Units.gridUnit
     Layout.minimumHeight: Kirigami.Units.gridUnit

     contentItem: ColumnLayout {
```
`minimumHeight` may also needs such resizing.

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

Reply via email to