[vlc-commits] [Git][videolan/vlc][master] qml: do not disable sub-pixel alignment with Qt < 6.2.5 in flickables

[Steve Lhomme (@robUx4)]

Steve Lhomme pushed to branch master at VideoLAN / VLC


Commits:
fe7adb23 by Fatih Uzunoglu at 2025-09-08T14:15:28+00:00
qml: do not disable sub-pixel alignment with Qt < 6.2.5 in flickables

This is for the Qt bug QTBUG-103996. Normally we should not care about
this because Qt 6.2.12 is already released open-source, but Qt does
not seem to offer these LTS versions pre-built and Ubuntu 20.04 LTS
still seems to offer Qt 6.2.4, so it can be useful to have this check
for now.

- - - - -


4 changed files:

- modules/gui/qt/medialibrary/qml/HomePage.qml
- modules/gui/qt/network/qml/BrowseHomeDisplay.qml
- modules/gui/qt/widgets/qml/ExpandGridView.qml
- modules/gui/qt/widgets/qml/ListViewExt.qml


Changes:

=====================================
modules/gui/qt/medialibrary/qml/HomePage.qml
=====================================
@@ -93,7 +93,8 @@ T.Page {
         topMargin: VLCStyle.layoutTitle_top_padding
         bottomMargin: topMargin
 
-        pixelAligned: (Screen.pixelDensity >= VLCStyle.highPixelDensityThreshold) // no need for sub-pixel alignment with high pixel density
+        pixelAligned: (MainCtx.qtVersion() >= MainCtx.qtVersionCheck(6, 2, 5)) // QTBUG-103996
+                      && (Screen.pixelDensity >= VLCStyle.highPixelDensityThreshold) // no need for sub-pixel alignment with high pixel density
 
         property bool _hasMedias: true
 


=====================================
modules/gui/qt/network/qml/BrowseHomeDisplay.qml
=====================================
@@ -134,7 +134,8 @@ FocusScope {
 
         focus: true
 
-        pixelAligned: (Screen.pixelDensity >= VLCStyle.highPixelDensityThreshold) // no need for sub-pixel alignment with high pixel density
+        pixelAligned: (MainCtx.qtVersion() >= MainCtx.qtVersionCheck(6, 2, 5)) // QTBUG-103996
+                      && (Screen.pixelDensity >= VLCStyle.highPixelDensityThreshold) // no need for sub-pixel alignment with high pixel density
 
         contentWidth: column.width
         contentHeight: column.height


=====================================
modules/gui/qt/widgets/qml/ExpandGridView.qml
=====================================
@@ -795,7 +795,8 @@ FocusScope {
 
         boundsBehavior: Flickable.StopAtBounds
 
-        pixelAligned: (Screen.pixelDensity >= VLCStyle.highPixelDensityThreshold) // no need for sub-pixel alignment with high pixel density
+        pixelAligned: (MainCtx.qtVersion() >= MainCtx.qtVersionCheck(6, 2, 5)) // QTBUG-103996
+                      && (Screen.pixelDensity >= VLCStyle.highPixelDensityThreshold) // no need for sub-pixel alignment with high pixel density
 
         ScrollBar.vertical: ScrollBarExt {
             id: flickableScrollBar


=====================================
modules/gui/qt/widgets/qml/ListViewExt.qml
=====================================
@@ -77,7 +77,8 @@ ListView {
 
     // Settings
 
-    pixelAligned: (Screen.pixelDensity >= VLCStyle.highPixelDensityThreshold) // no need for sub-pixel alignment with high pixel density
+    pixelAligned: (MainCtx.qtVersion() >= MainCtx.qtVersionCheck(6, 2, 5)) // QTBUG-103996
+                  && (Screen.pixelDensity >= VLCStyle.highPixelDensityThreshold) // no need for sub-pixel alignment with high pixel density
 
     focus: true
 



View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/fe7adb23cd4ee690067c0b428d1007ce565e9dfe

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/fe7adb23cd4ee690067c0b428d1007ce565e9dfe
You're receiving this email because of your account on code.videolan.org.


VideoLAN code repository instance