Changeset 71191f1 in subsurface


Ignore:
Timestamp:
Apr 13, 2017, 11:05:26 AM (2 months ago)
Author:
Dirk Hohndel <dirk@…>
Branches:
master
Children:
241bd9a
Parents:
a0c6cf5
git-author:
Marco Martin <notmart@…> (04/13/17 10:57:22)
git-committer:
Dirk Hohndel <dirk@…> (04/13/17 11:05:26)
Message:

QML UI: only assign actions in a declarative way

Assigning actions in an imperative way on component complete,
seems to cause a crash on some devices, assign declaratively
the whole list instead, hiding the gps related actions on
iOS as they are not supported on that platform.

Signed-off-by: Marco Martin <notmart@…>
Signed-off-by: Dirk Hohndel <dirk@…>

File:
1 edited

Legend:

Unmodified
Added
Removed
  • mobile-widgets/qml/main.qml

    r751626e r71191f1  
    107107                bannerImageSource: "dive.jpg"
    108108
    109                 property list<QtObject> topActions: [
     109                actions: [
    110110                        Kirigami.Action {
    111111                                text: qsTr("Dive list")
     
    181181                                        }
    182182                                }
    183                         }
    184                 ] // end topActions
    185 
    186                 property list<QtObject> gpsActions: [
     183                        },
    187184                        Kirigami.Action {
    188185                                text: qsTr("GPS")
    189186                                enabled: manager.credentialStatus === QMLManager.VALID || manager.credentialStatus === QMLManager.VALID_EMAIL
     187                                visible: (Qt.platform.os !== "ios")
    190188                                Kirigami.Action {
    191189                                        text: qsTr("GPS-tag dives")
     
    231229                                        }
    232230                                }
    233                         }
    234                 ] // end gpsActions
    235 
    236                 property list<QtObject> bottomActions: [
     231                        },
    237232                        Kirigami.Action {
    238233                                text: qsTr("Developer")
     
    264259                                }
    265260                        }
    266                 ] // end bottonActions
    267 
    268                 Component.onCompleted: {
    269                         var createActions = new Array(0)
    270                         for (var i = 0; i < topActions.length; i++)
    271                                 createActions.push(topActions[i])
    272                         if (Qt.platform.os !== "ios") {
    273                                 for (var i = 0; i < gpsActions.length; i++)
    274                                         createActions.push(gpsActions[i])
    275                         }
    276                         for (var i = 0; i < bottomActions.length; i++)
    277                                 createActions.push(bottomActions[i])
    278                         actions = createActions
    279                         print(actions)
    280                 }
     261                ] // end actions
    281262
    282263                MouseArea {
Note: See TracChangeset for help on using the changeset viewer.