[frameworks/qqc2-desktop-style] org.kde.desktop: TableHeaderViews: account for the margins of the root control

Ismael Asensio <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 893cc25c424ca0552fc086568ee9a7f946447df5 by Ismael Asensio.
Committed on 16/08/2026 at 11:31.
Pushed by iasensio into branch 'master'.

TableHeaderViews: account for the margins of the root control

The template of the `TableHeaderView` places an element after the last
column/row of the header to cover the space after the delegates.

Fix it so it also accounts for the control margins, so it doesn't
overdraw out of place

CCBUG: 524099

M  +2    -1    org.kde.desktop/HorizontalHeaderView.qml
M  +2    -1    org.kde.desktop/VerticalHeaderView.qml

https://invent.kde.org/frameworks/qqc2-desktop-style/-/commit/893cc25c424ca0552fc086568ee9a7f946447df5

diff --git a/org.kde.desktop/HorizontalHeaderView.qml b/org.kde.desktop/HorizontalHeaderView.qml
index 20393afc..05008c9d 100644
--- a/org.kde.desktop/HorizontalHeaderView.qml
+++ b/org.kde.desktop/HorizontalHeaderView.qml
@@ -70,7 +70,8 @@ T.HorizontalHeaderView {
     StylePrivate.StyleItem {
         parent: controlRoot
         anchors.fill: parent
-        anchors.leftMargin: controlRoot.contentWidth
+        anchors.leftMargin: controlRoot.contentWidth + controlRoot.leftMargin
+        anchors.rightMargin: controlRoot.rightMargin
         z: -1
         elementType: "header"
         properties: {
diff --git a/org.kde.desktop/VerticalHeaderView.qml b/org.kde.desktop/VerticalHeaderView.qml
index f45c5fbc..d9d50860 100644
--- a/org.kde.desktop/VerticalHeaderView.qml
+++ b/org.kde.desktop/VerticalHeaderView.qml
@@ -62,7 +62,8 @@ T.VerticalHeaderView {
     StylePrivate.StyleItem {
         parent: controlRoot
         anchors.fill: parent
-        anchors.topMargin: controlRoot.contentHeight
+        anchors.topMargin: controlRoot.contentHeight + controlRoot.topMargin
+        anchors.bottomMargin: controlRoot.bottomMargin
         z: -1
         elementType: "header"
         properties: {
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.