MerginMaps · kaustuvpokharel · Oct 10, 2025 · Oct 10, 2025 · Oct 18, 2025 · Oct 22, 2025
diff --git a/app/qml/components/MMListDelegate.qml b/app/qml/components/MMListDelegate.qml
@@ -40,7 +40,7 @@ Item {
 
   property real verticalSpacing: root.secondaryText ? __style.margin8 : __style.margin20
 
-  implicitWidth: ListView?.view?.width ?? 0 // in case ListView is injected as attached property (usually it is)
+  implicitWidth: ListView?.view?.width - ListView?.view?.scrollBarWidth ?? 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 ¯\_(ツ)_/¯
 

diff --git a/app/qml/components/MMListView.qml b/app/qml/components/MMListView.qml
@@ -8,13 +8,27 @@
  ***************************************************************************/
 
 import QtQuick
+import QtQuick.Controls
+import QtQml
 
 //
 // Hot-fix for hotfix https://github.com/MerginMaps/mobile/issues/3417
 // Seems like there is some issue with cache in ListView
 //
 
 ListView {
+  id: root
 
   cacheBuffer: 0
+  readonly property bool isMobile: (Qt.platform.os === "android" || Qt.platform.os === "ios")
+  readonly property int scrollBarWidth: !isMobile ? __style.margin10 : 0
+  property alias showScrollBar: verticalScrollBar.policy
+
+  ScrollBar.vertical: ScrollBar{
+    id: verticalScrollBar
+
+    policy: isMobile ? ScrollBar.AlwaysOff : ScrollBar.AlwaysOn
+    visible: contentHeight > availableHeight ? true  : false
+    opacity: active ? 0.7 : 0.4
+  }
 }
diff --git a/app/qml/components/MMScrollView.qml b/app/qml/components/MMScrollView.qml
@@ -6,18 +6,25 @@
  *   (at your option) any later version.                                   *
  *                                                                         *
  ***************************************************************************/
-
 import QtQuick
 import QtQuick.Controls
-
-// Convenient class to use as a pageContent or drawerContent
-// base element to make the content scroll
+import QtQml
 
 ScrollView {
-  id: root
+    id: root
+
+    readonly property bool isMobile: (Qt.platform.os === "android"
+                                      || Qt.platform.os === "ios")
+    readonly property int scrollBarWidth: !isMobile ? __style.margin10 : 0
+    property bool showScrollBar:  root.ScrollBar.vertical.policy
+
+
+    contentWidth: availableWidth - scrollBarWidth
 
-  contentWidth: availableWidth // to only scroll vertically
+    ScrollBar.horizontal.policy: ScrollBar.AlwaysOff
 
-  ScrollBar.vertical.policy: ScrollBar.AlwaysOff
-  ScrollBar.horizontal.policy: ScrollBar.AlwaysOff
+    // vertical scroll bar should appear only when needed
+    ScrollBar.vertical.policy: isMobile ? ScrollBar.AlwaysOff : ScrollBar.AlwaysOn
+    ScrollBar.vertical.visible: contentHeight > availableHeight ? true  : false
+    ScrollBar.vertical.opacity: active ? 0.7 : 0.4
 }
diff --git a/app/qml/components/MMToolbar.qml b/app/qml/components/MMToolbar.qml
@@ -8,6 +8,7 @@
  ***************************************************************************/
 
 import QtQuick
+import QtQuick.Controls
 
 import "./private"
 
@@ -43,6 +44,7 @@ Rectangle {
 
     MMListView {
       id: toolbar
+      showScrollBar : ScrollBar.AlwaysOff
 
       onWidthChanged: root.recalculate()
       model: toolbarModel

diff --git a/app/qml/form/MMFormPage.qml b/app/qml/form/MMFormPage.qml
@@ -276,7 +276,7 @@ Page {
 
     Item {
 
-      width: ListView.view.width
+      width: ListView.view.width - ListView.view.scrollBarWidth
       implicitHeight: childrenRect.height
 
       // In future, better to filter such fields in the field proxy model instead

diff --git a/app/qml/form/MMPreviewDrawer.qml b/app/qml/form/MMPreviewDrawer.qml
@@ -193,7 +193,7 @@ Item {
           height: contentHeight
 
           spacing: __style.margin8
-          interactive: false
+          showScrollBar: ScrollBar.AsNeeded
 
           model: root.controller.fieldModel
 

diff --git a/app/qml/layers/MMLayerDetailPage.qml b/app/qml/layers/MMLayerDetailPage.qml
@@ -193,8 +193,6 @@ Page {
                     wrapMode: Text.WordWrap
                     text: __inputUtils.layerAttribution(layerDetailData.mapLayer)
                   }
-
-                  ScrollBar.vertical: ScrollBar {}
                 }
               }
             }

diff --git a/app/qml/project/MMProjectList.qml b/app/qml/project/MMProjectList.qml
@@ -94,7 +94,7 @@ Item {
     delegate: MMProjectComponents.MMProjectDelegate {
       id: projectDelegate
 
-      width: ListView.view.width
+      width: ListView.view.width - ListView.view.scrollBarWidth
       height: visible ? implicitHeight : 0
 
       projectDisplayName: root.projectModelType === MM.ProjectsModel.WorkspaceProjectsModel ? model.ProjectName : model.ProjectFullName
@@ -202,7 +202,7 @@ Item {
 
       topPadding: noLocalProjectsMessageContainer.visible ? noLocalProjectsMessageContainer.height + __style.margin40 : 0
 
-      width: ListView.view.width
+      width: ListView.view.width - ListView.view.scrollBarWidth
 
       Item {
         width: parent.width

diff --git a/app/qml/project/MMProjectWizardPage.qml b/app/qml/project/MMProjectWizardPage.qml
@@ -70,7 +70,7 @@ MMComponents.MMPage {
         delegate: MMProjectComponents.MMProjectWizardDelegate {
           id: fieldDelegate
 
-          width: ListView.view.width
+          width: ListView.view.width - ListView.view.scrollBarWidth
 
           // find current index in the model
           comboboxField.comboboxModel: typesmodel

diff --git a/app/qml/project/components/MMProjectDelegate.qml b/app/qml/project/components/MMProjectDelegate.qml
@@ -37,6 +37,7 @@ Control {
   signal showChangesRequested()
 
   height: implicitHeight
+  width: ListView.view.width - ListView.view.scrollBarWidth
 
   topPadding: __style.margin20
   rightPadding: __style.margin20

diff --git a/app/qml/settings/MMLogPage.qml b/app/qml/settings/MMLogPage.qml
@@ -53,7 +53,15 @@ MMPage {
         width: parent.width
       }
 
-      ScrollBar.vertical: ScrollBar { }
+      ScrollBar.vertical: ScrollBar {
+        opacity: active ? 0.7 : 0.4
+
+        contentItem: Rectangle {
+          implicitWidth: 5
+          radius: width / 2
+          color: __style.darkGreenColor
+        }
+      }
     }
 
     MMButton {
-Original file line number
+Diff line change
@@ Expand Up / @@ -193,8 +193,6 @@ Page { @@
                         wrapMode: Text.WordWrap
                         text: __inputUtils.layerAttribution(layerDetailData.mapLayer)
                       }
-                      ScrollBar.vertical: ScrollBar {}
                     }
                   }
                 }
@@ Expand Down @@