Add missing scrollbar in variables list, tweak padding

opengisch · Aug 27, 2024 · ef6754d · ef6754d
1 parent 81b05a7
commit ef6754d
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/src/qml/VariableEditor.qml b/src/qml/VariableEditor.qml
@@ -34,6 +34,9 @@ ColumnLayout {
  spacing: 1
  anchors.fill: parent
 
+ ScrollBar.vertical: QfScrollBar {
+ }
+
  model: ExpressionVariableModel {
  currentProject: qgisProject
  }
@@ -76,15 +79,15 @@ ColumnLayout {
  id: line
  anchors {
  left: parent.left
- leftMargin: 5
+ leftMargin: 10
  right: parent.right
- rightMargin: 5
+ rightMargin: 10
  }
  spacing: 0
 
  QfSwipeAnimator {
  id: variableNameTextAnimator
- width: table.width - 10
+ width: table.width - line.anchors.leftMargin * 2
  height: 24
  shouldAutoFlick: (width < variableNameText.implicitWidth) && !dragging && !variableNameText.activeFocus
  contentImplicitWidth: variableNameText.implicitWidth
@@ -93,11 +96,11 @@ ColumnLayout {
 
  TextField {
  id: variableNameText
- topPadding: 0
+ topPadding: 5
  bottomPadding: 0
  leftPadding: 1
  rightPadding: 1
- width: Math.max(table.width - 10, implicitWidth)
+ width: Math.max(table.width - line.anchors.leftMargin * 2, implicitWidth)
  height: variableNameTextAnimator.height
  text: VariableName
  enabled: VariableScope == ExpressionVariableModel.GlobalScope && VariableEditable
@@ -127,7 +130,7 @@ ColumnLayout {
 
  QfSwipeAnimator {
  id: variableValueTextAnimator
- width: table.width - 10 - (canDelete ? deleteVariableButton.width : 0)
+ width: table.width - line.anchors.leftMargin * 2 - (canDelete ? deleteVariableButton.width : 0)
  height: 40
  shouldAutoFlick: (width < variableValueText.implicitWidth) && !dragging && !variableValueText.activeFocus
  contentImplicitWidth: variableValueText.implicitWidth
@@ -140,7 +143,7 @@ ColumnLayout {
  bottomPadding: 10
  leftPadding: 1
  rightPadding: 1
- width: Math.max(table.width - 10 - (canDelete ? deleteVariableButton.width : 0), implicitWidth)
+ width: Math.max(table.width - line.anchors.leftMargin * 2 - (canDelete ? deleteVariableButton.width : 0), implicitWidth)
  text: VariableValue
  enabled: VariableEditable
  font: Theme.defaultFont