Skip to content

Commit

Permalink
Fix inconsistent UI/UX for read-only non-spatial layers (#3635)
Browse files Browse the repository at this point in the history
* wip - enhancing ui/ux

* adding layerIsReadOnly condition to add feature button to be visible

* using current layer property
  • Loading branch information
VitorVieiraZ authored Oct 8, 2024
1 parent 153c496 commit b2eff70
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion app/qml/layers/MMFeaturesListPage.qml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ MMComponents.MMPage {

property var selectedLayer: null
property bool hasToolbar: false
property bool layerIsReadOnly: selectedLayer?.readOnly ?? false

signal featureClicked( var featurePair )
signal addFeatureClicked( var toLayer )
Expand Down Expand Up @@ -91,7 +92,7 @@ MMComponents.MMPage {
anchors.bottom: parent.bottom
anchors.bottomMargin: root.hasToolbar ? __style.margin20 : ( __style.safeAreaBottom + __style.margin8 )

visible: __inputUtils.isNoGeometryLayer( root.selectedLayer )
visible: __inputUtils.isNoGeometryLayer( root.selectedLayer ) && !root.layerIsReadOnly

text: qsTr("Add feature")

Expand Down

1 comment on commit b2eff70

@inputapp-bot
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

iOS - version 24.10.668511 just submitted!

Please sign in to comment.