fixed suggestions

Author: kaustuvpokharel

app/qml/components/MMListDelegate.qml

@@ -24,7 +24,6 @@ Item {
   property string text
   property string secondaryText
 
-  readonly property int scrollSpace: (ListView.view && ListView.view.scrollSpace) ? ListView.view.scrollSpace : 0
   property alias leftContent: leftContentGroup.children
   property alias rightContent: rightContentGroup.children
 
@@ -44,7 +43,7 @@ Item {
   implicitWidth: ListView?.view?.width ?? 0  // in case ListView is injected as attached property (usually it is)
   implicitHeight: contentLayout.implicitHeight
   height: visible ? implicitHeight : 0.1 // hide invisible items, for some reason setting 0 does not work ¯\_(ツ)_/¯
-  width: implicitWidth - ListView.view.scrollSpace
+  width: implicitWidth - ( ListView?.view?.scrollSpace ?? 0 )
 
   MouseArea {
     anchors.fill: contentLayout

app/qml/components/MMListView.qml

@@ -9,6 +9,7 @@
 
 import QtQuick
 import QtQuick.Controls
+import QtQml
 
 //
 // Hot-fix for hotfix https://github.com/MerginMaps/mobile/issues/3417
@@ -29,10 +30,6 @@ ListView {
     visible: !isMobile && (root.contentHeight > root.height)
     opacity: (pressed || root.moving) ? 0.7 : 0.4
 
-    anchors.right: root.right
-
-    implicitHeight: root.height
-
     contentItem: Rectangle {
       implicitWidth: 5
       radius: width / 2

app/qml/components/MMScrollView.qml

@@ -9,6 +9,7 @@
 
 import QtQuick
 import QtQuick.Controls
+import QtQml
 
 // Convenient class to use as a pageContent or drawerContent
 // base element to make the content scroll
@@ -19,26 +20,22 @@ ScrollView {
   readonly property bool isMobile: (Qt.platform.os === "android" || Qt.platform.os === "ios")
   property int scrollSpace: !isMobile ? 10 : 0
 
-  contentWidth: availableWidth - scrollSpace // to only scroll vertically
-
-  ScrollBar.horizontal: null
+  contentWidth: availableWidth - scrollSpace
 
+  ScrollBar.horizontal.policy: ScrollBar.AlwaysOff
   ScrollBar.vertical: ScrollBar {
     id: verticalScrollBar
 
-    policy: ScrollBar.AsNeeded
+    policy: ScrollBar.AlwaysOn
     visible: !isMobile && (root.contentHeight > root.height)
     opacity: (pressed || root.moving) ? 0.7 : 0.4
-
-    anchors.right: root.right
-
     implicitHeight: root.height
+    anchors.right: parent.right
 
     contentItem: Rectangle {
       implicitWidth: 5
       radius: width / 2
       color: __style.darkGreenColor
     }
   }
-
 }

app/qml/form/MMPreviewDrawer.qml

@@ -198,7 +198,7 @@ Item {
           model: root.controller.fieldModel
 
           delegate: Item {
-            width: ListView.view.width - ListView.view.width
+            width: ListView.view.width - ListView.view.scrollSpace
             height: childrenRect.height
 
             Column {

app/qml/layers/MMFeaturesListPage.qml

@@ -10,7 +10,6 @@
 import QtQuick
 import QtQuick.Controls
 import QtQuick.Layouts
-import QtQml
 
 import mm 1.0 as MM
 

app/qml/project/MMProjectWizardPage.qml

@@ -70,7 +70,7 @@ MMComponents.MMPage {
         delegate: MMProjectComponents.MMProjectWizardDelegate {
           id: fieldDelegate
 
-          width: ListView.view.width - ListView.view.width
+          width: ListView.view.width - ListView.view.scrollSpace
 
           // find current index in the model
           comboboxField.comboboxModel: typesmodel

app/qml/settings/MMLogPage.qml

@@ -53,7 +53,15 @@ MMPage {
         width: parent.width
       }
 
-      ScrollBar.vertical: ScrollBar { }
+      ScrollBar.vertical: ScrollBar {
+        opacity: (root.pressed || root.moving) ? 0.7 : 0.4
+
+        contentItem: Rectangle {
+          implicitWidth: 5
+          radius: width / 2
+          color: __style.darkGreenColor
+        }
+      }
     }
 
     MMButton {