ui: clean up QML 87/25887/2
authorGiWoong Kim <giwoong.kim@samsung.com>
Tue, 12 Aug 2014 04:12:19 +0000 (13:12 +0900)
committerGiWoong Kim <giwoong.kim@samsung.com>
Tue, 12 Aug 2014 07:47:35 +0000 (16:47 +0900)
delete a QtQuickItem
remove warnings
modified varable & function names

Change-Id: Ic8044786c7f7e3163e16503c3408fc441ce992a8
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>
17 files changed:
tizen/src/display/qt5_supplement.cpp
tizen/src/ui/mainwindow.cpp
tizen/src/ui/resource/mobile-720x1280-3btn/layout.qml
tizen/src/ui/resource/wearable-320x320-1btn/layout.qml
tizen/src/ui/xml/Makefile.objs
tizen/src/ui/xml/emulatoruitype.cpp [new file with mode: 0644]
tizen/src/ui/xml/emulatoruitype.h [new file with mode: 0644]
tizen/src/ui/xml/formlisttype.cpp [new file with mode: 0644]
tizen/src/ui/xml/formlisttype.h [new file with mode: 0644]
tizen/src/ui/xml/formtype.cpp
tizen/src/ui/xml/formtype.h
tizen/src/ui/xml/hardwarekeytype.cpp
tizen/src/ui/xml/hardwarekeytype.h
tizen/src/ui/xml/keylisttype.cpp
tizen/src/ui/xml/keylisttype.h
tizen/src/ui/xml/regiontype.cpp
tizen/src/ui/xml/regiontype.h

index 07dce80336e43ee18dc7b0a0f84982aef82a704d..9195e30bd526600e8178c2791b12ebbe33934683 100644 (file)
  */
 
 #include <QApplication>
-#include "qt5_supplement.h"
-
 #include <QQmlComponent>
 #include <QQmlEngine>
 #include <QQmlProperty>
-#include <QtQuick/QQuickItem>
 
+#include "qt5_supplement.h"
 #include "mainwindow.h"
 #include "uiinformation.h"
 #include "hardwarekey.h"
 #include "controllerwidget.h"
-#include "ui/xml/hardwarekeytype.h"
-#include "ui/xml/keylisttype.h"
-#include "ui/xml/formtype.h"
+#include "ui/xml/emulatoruitype.h"
 
 extern "C" {
 #include "emul_state.h"
@@ -198,6 +194,8 @@ void loadSkinFormFromXML(QFile *file, UIInformation *uiInfo/* out */)
 
     qDebug("skin form is loaded from %s", qPrintable(file->fileName()));
 
+    qmlRegisterType<EmulatorUIType>("EmulatorComponent", 1, 0, "EmulatorUI");
+    qmlRegisterType<FormListType>("EmulatorComponent", 1, 0, "FormList");
     qmlRegisterType<FormType>("EmulatorComponent", 1, 0, "Form");
 
     /* QML */
@@ -214,17 +212,18 @@ void loadSkinFormFromXML(QFile *file, UIInformation *uiInfo/* out */)
         QFileInfo fileInfo(*file);
         qDebug() << "xml version :" << QQmlProperty::read(object, "version").toString();
 
+        EmulatorUIType *uiType = (EmulatorUIType *)object;
         FormType *formType = NULL;
         KeyListType *keyListType = NULL;
         HardwareKeyType *hwKeyType = NULL;
 
-        QObjectList formTypeList = object->children();
-        for (int index = 0; index < formTypeList.count(); index++) {
+        FormListType *formListType = uiType->formListType();
+        for (int index = 0; index < formListType->getFormList()->count(); index++) {
             SkinForm *form = new SkinForm(); /* dst */
 
-            formType = (FormType *)formTypeList.at(index); /* src */
+            formType = (FormType *)formListType->getFormList()->at(index); /* src */
 
-            form->displayRegion = formType->displayRegion();
+            form->displayRegion = formType->getDisplayRegion();
             form->skinImg[SkinForm::normal].load(
                         fileInfo.absolutePath() + "/" + formType->mainImageName());
             form->skinImg[SkinForm::pressed].load(
@@ -232,12 +231,11 @@ void loadSkinFormFromXML(QFile *file, UIInformation *uiInfo/* out */)
 
             keyListType = formType->keyListType();
             if (keyListType != NULL) {
-                for (int i = 0; i < keyListType->list.count(); i++) {
-                    hwKeyType = keyListType->list.at(i);
+                for (int i = 0; i < keyListType->getKeyList()->count(); i++) {
+                    hwKeyType = keyListType->getKeyList()->at(i);
                     if (hwKeyType != NULL) {
-                        form->keyList.append(
-                                    new HardwareKey(hwKeyType->objectName(), hwKeyType->keycode(),
-                                                    hwKeyType->region(), hwKeyType->tooltip()));
+                        form->keyList.append(new HardwareKey(hwKeyType->objectName(),
+                            hwKeyType->keycode(), hwKeyType->region(), hwKeyType->tooltip()));
                     }
                 }
             }
@@ -287,12 +285,11 @@ void loadControllerFormFromXML(QFile *file, UIInformation *uiInfo/* out */)
 
         keyListType = formType->keyListType();
         if (keyListType != NULL) {
-            for (int i = 0; i < keyListType->list.count(); i++) {
-                hwKeyType = keyListType->list.at(i);
+            for (int i = 0; i < keyListType->getKeyList()->count(); i++) {
+                hwKeyType = keyListType->getKeyList()->at(i);
                 if (hwKeyType != NULL) {
-                    form->keyList.append(
-                                new HardwareKey(hwKeyType->objectName(), hwKeyType->keycode(),
-                                                hwKeyType->region(), hwKeyType->tooltip()));
+                    form->keyList.append(new HardwareKey(hwKeyType->objectName(),
+                        hwKeyType->keycode(), hwKeyType->region(), hwKeyType->tooltip()));
                 }
             }
         }
index 3a025ee5a3555e0cac1a5877a18ed0d64858a2fd..63dee5f1c35ca17c45305437248afddd74e078c1 100644 (file)
@@ -71,8 +71,11 @@ MainWindow::MainWindow(UIInformation *uiInfo, QWidget *parent) :
     QHBoxLayout *winLayout = new QHBoxLayout(this);
     winLayout->setMargin(0);
     winLayout->setSpacing(0);
+
     /* scene */
     QGraphicsScene* skinScene = new QGraphicsScene(this);
+    //skinScene->setBackgroundBrush(Qt::black);
+
     /* view */
     skinView = new SkinView(skinScene, this);
     winLayout->addWidget(skinView);
index ab67ef49a1d4a7175e312b2ce705f3069579143e..012075db3d8290886dd391bc2b43fe8ff0cfcd64 100644 (file)
-import QtQuick 2.0
 import EmulatorComponent 1.0
 
-Item {
+EmulatorUI {
     property string version: "1.0"
 
-    Form {
-        objectName: "portrait"
-        display: Region { left: 67; top: 116; width: 720; height: 1280 }
+    FormList {
+        Form {
+            objectName: "portrait"
+            display: Region { left: 67; top: 116; width: 720; height: 1280 }
 
-        mainImage: "default_0.png"
-        pressedImage: "default_0_p.png"
+            mainImage: "default_0.png"
+            pressedImage: "default_0_p.png"
 
-        KeyList {
-            Key { objectName: "Menu"; keycode: 169
-                Region { left: 210; top: 1401; width: 84; height: 84 }
-                tooltip: "Menu"
-            }
-            Key { objectName: "Home"; keycode: 139
-                Region { left: 390; top: 1401; width: 84; height: 84 }
-                tooltip: "Home"
-            }
-            Key { objectName: "Back"; keycode: 158
-                Region { left: 570; top: 1401; width: 84; height: 84 }
-                tooltip: "Back"
-            }
-            Key { objectName: "Power"; keycode: 116
-                Region { left: 841; top: 1309; width: 24; height: 96 }
-                tooltip: "Power"
-            }
-            Key { objectName: "Volume +"; keycode: 115
-                Region { left: 841; top: 108; width: 24; height: 96 }
-                tooltip: "Volume +"
-            }
-            Key { objectName: "Volume -"; keycode: 114
-                Region { left: 841; top: 219; width: 24; height: 96 }
-                tooltip: "Volume -"
+            KeyList {
+                Key { objectName: "Menu"; keycode: 169
+                    Region { left: 210; top: 1401; width: 84; height: 84 }
+                    tooltip: "Menu"
+                }
+                Key { objectName: "Home"; keycode: 139
+                    Region { left: 390; top: 1401; width: 84; height: 84 }
+                    tooltip: "Home"
+                }
+                Key { objectName: "Back"; keycode: 158
+                    Region { left: 570; top: 1401; width: 84; height: 84 }
+                    tooltip: "Back"
+                }
+                Key { objectName: "Power"; keycode: 116
+                    Region { left: 841; top: 1309; width: 24; height: 96 }
+                    tooltip: "Power"
+                }
+                Key { objectName: "Volume +"; keycode: 115
+                    Region { left: 841; top: 108; width: 24; height: 96 }
+                    tooltip: "Volume +"
+                }
+                Key { objectName: "Volume -"; keycode: 114
+                    Region { left: 841; top: 219; width: 24; height: 96 }
+                    tooltip: "Volume -"
+                }
             }
         }
-    }
 
-    Form {
-        objectName: "landscape"
-        display: Region { left: 116; top: 78; width: 1280; height: 720 }
+        Form {
+            objectName: "landscape"
+            display: Region { left: 116; top: 78; width: 1280; height: 720 }
 
-        mainImage: "default_L90.png"
-        pressedImage: "default_L90_p.png"
+            mainImage: "default_L90.png"
+            pressedImage: "default_L90_p.png"
 
-        KeyList {
-            Key { objectName: "Menu"; keycode: 169
-                Region { left: 1400; top: 570; width: 84; height: 84 }
-                tooltip: "Menu"
-            }
-            Key { objectName: "Home"; keycode: 139
-                Region { left: 1400; top: 392; width: 84; height: 84 }
-                tooltip: "Home"
-            }
-            Key { objectName: "Back"; keycode: 158
-                Region { left: 1400; top: 210; width: 84; height: 84 }
-                tooltip: "Back"
-            }
-            Key { objectName: "Power"; keycode: 116
-                Region { left: 1308; top: 2; width: 96; height: 24 }
-                tooltip: "Power"
-            }
-            Key { objectName: "Volume +"; keycode: 115
-                Region { left: 108; top: 2; width: 96; height: 24 }
-                tooltip: "Volume +"
-            }
-            Key { objectName: "Volume -"; keycode: 114
-                Region { left: 217; top: 2; width: 96; height: 24 }
-                tooltip: "Volume -"
+            KeyList {
+                Key { objectName: "Menu"; keycode: 169
+                    Region { left: 1400; top: 570; width: 84; height: 84 }
+                    tooltip: "Menu"
+                }
+                Key { objectName: "Home"; keycode: 139
+                    Region { left: 1400; top: 392; width: 84; height: 84 }
+                    tooltip: "Home"
+                }
+                Key { objectName: "Back"; keycode: 158
+                    Region { left: 1400; top: 210; width: 84; height: 84 }
+                    tooltip: "Back"
+                }
+                Key { objectName: "Power"; keycode: 116
+                    Region { left: 1308; top: 2; width: 96; height: 24 }
+                    tooltip: "Power"
+                }
+                Key { objectName: "Volume +"; keycode: 115
+                    Region { left: 108; top: 2; width: 96; height: 24 }
+                    tooltip: "Volume +"
+                }
+                Key { objectName: "Volume -"; keycode: 114
+                    Region { left: 217; top: 2; width: 96; height: 24 }
+                    tooltip: "Volume -"
+                }
             }
         }
-    }
 
-    Form {
-        objectName: "reverse portrait"
-        display: Region { left: 78; top: 117; width: 720; height: 1280 }
+        Form {
+            objectName: "reverse portrait"
+            display: Region { left: 78; top: 117; width: 720; height: 1280 }
 
-        mainImage: "default_180.png"
-        pressedImage: "default_180_p.png"
+            mainImage: "default_180.png"
+            pressedImage: "default_180_p.png"
 
-        KeyList {
-            Key { objectName: "Menu"; keycode: 169
-                Region { left: 570; top: 30; width: 84; height: 84 }
-                tooltip: "Menu"
-            }
-            Key { objectName: "Home"; keycode: 139
-                Region { left: 390; top: 30; width: 84; height: 84 }
-                tooltip: "Home"
-            }
-            Key { objectName: "Back"; keycode: 158
-                Region { left: 210; top: 30; width: 84; height: 84 }
-                tooltip: "Back"
-            }
-            Key { objectName: "Power"; keycode: 116
-                Region { left: 2; top: 110; width: 24; height: 96 }
-                tooltip: "Power"
-            }
-            Key { objectName: "Volume +"; keycode: 115
-                Region { left: 2; top: 1310; width: 24; height: 96 }
-                tooltip: "Volume +"
-            }
-            Key { objectName: "Volume -"; keycode: 114
-                Region { left: 2; top: 1200; width: 24; height: 96 }
-                tooltip: "Volume -"
+            KeyList {
+                Key { objectName: "Menu"; keycode: 169
+                    Region { left: 570; top: 30; width: 84; height: 84 }
+                    tooltip: "Menu"
+                }
+                Key { objectName: "Home"; keycode: 139
+                    Region { left: 390; top: 30; width: 84; height: 84 }
+                    tooltip: "Home"
+                }
+                Key { objectName: "Back"; keycode: 158
+                    Region { left: 210; top: 30; width: 84; height: 84 }
+                    tooltip: "Back"
+                }
+                Key { objectName: "Power"; keycode: 116
+                    Region { left: 2; top: 110; width: 24; height: 96 }
+                    tooltip: "Power"
+                }
+                Key { objectName: "Volume +"; keycode: 115
+                    Region { left: 2; top: 1310; width: 24; height: 96 }
+                    tooltip: "Volume +"
+                }
+                Key { objectName: "Volume -"; keycode: 114
+                    Region { left: 2; top: 1200; width: 24; height: 96 }
+                    tooltip: "Volume -"
+                }
             }
         }
-    }
 
-    Form {
-        objectName: "reverse landscape"
-        display: Region { left: 117; top: 67; width: 1280; height: 720 }
+        Form {
+            objectName: "reverse landscape"
+            display: Region { left: 117; top: 67; width: 1280; height: 720 }
 
-        mainImage: "default_R90.png"
-        pressedImage: "default_R90_p.png"
+            mainImage: "default_R90.png"
+            pressedImage: "default_R90_p.png"
 
-        KeyList {
-            Key { objectName: "Menu"; keycode: 169
-                Region { left: 28; top: 209; width: 84; height: 84 }
-                tooltip: "Menu"
-            }
-            Key { objectName: "Home"; keycode: 139
-                Region { left: 28; top: 391; width: 84; height: 84 }
-                tooltip: "Home"
-            }
-            Key { objectName: "Back"; keycode: 158
-                Region { left: 28; top: 568; width: 84; height: 84 }
-                tooltip: "Back"
-            }
-            Key { objectName: "Power"; keycode: 116
-                Region { left: 109; top: 841; width: 96; height: 24 }
-                tooltip: "Power"
-            }
-            Key { objectName: "Volume +"; keycode: 115
-                Region { left: 1309; top: 841; width: 96; height: 24 }
-                tooltip: "Volume +"
-            }
-            Key { objectName: "Volume -"; keycode: 114
-                Region { left: 1199; top: 841; width: 96; height: 24 }
-                tooltip: "Volume -"
+            KeyList {
+                Key { objectName: "Menu"; keycode: 169
+                    Region { left: 28; top: 209; width: 84; height: 84 }
+                    tooltip: "Menu"
+                }
+                Key { objectName: "Home"; keycode: 139
+                    Region { left: 28; top: 391; width: 84; height: 84 }
+                    tooltip: "Home"
+                }
+                Key { objectName: "Back"; keycode: 158
+                    Region { left: 28; top: 568; width: 84; height: 84 }
+                    tooltip: "Back"
+                }
+                Key { objectName: "Power"; keycode: 116
+                    Region { left: 109; top: 841; width: 96; height: 24 }
+                    tooltip: "Power"
+                }
+                Key { objectName: "Volume +"; keycode: 115
+                    Region { left: 1309; top: 841; width: 96; height: 24 }
+                    tooltip: "Volume +"
+                }
+                Key { objectName: "Volume -"; keycode: 114
+                    Region { left: 1199; top: 841; width: 96; height: 24 }
+                    tooltip: "Volume -"
+                }
             }
         }
     }
index f7a0a1c853c936a0aac0163bfba7e540fd654e1f..9fee04421d3aac98bc2fc4aed097027cc58f9f5b 100644 (file)
@@ -1,65 +1,66 @@
-import QtQuick 2.0
 import EmulatorComponent 1.0
 
-Item {
+EmulatorUI {
     property string version: "1.0"
 
-    Form {
-        objectName: "portrait"
-        display: Region { left: 26; top: 60; width: 320; height: 320 }
+    FormList {
+        Form {
+            objectName: "portrait"
+            display: Region { left: 26; top: 60; width: 320; height: 320 }
 
-        mainImage: "default_0.png"
-        pressedImage: "default_0_p.png"
+            mainImage: "default_0.png"
+            pressedImage: "default_0_p.png"
 
-        KeyList {
-            Key { objectName: "Power"; keycode: 116
-                Region { left: 372; top: 116; width: 12; height: 80 }
-                tooltip: "Power"
+            KeyList {
+                Key { objectName: "Power"; keycode: 116
+                    Region { left: 372; top: 116; width: 12; height: 80 }
+                    tooltip: "Power"
+                }
             }
         }
-    }
 
-    Form {
-        objectName: "landscape"
-        display: Region { left: 60; top: 38; width: 320; height: 320 }
+        Form {
+            objectName: "landscape"
+            display: Region { left: 60; top: 38; width: 320; height: 320 }
 
-        mainImage: "default_L90.png"
-        pressedImage: "default_L90_p.png"
+            mainImage: "default_L90.png"
+            pressedImage: "default_L90_p.png"
 
-        KeyList {
-            Key { objectName: "Power"; keycode: 116
-                Region { left: 116; top: 0; width: 80; height: 12 }
-                tooltip: "Power"
+            KeyList {
+                Key { objectName: "Power"; keycode: 116
+                    Region { left: 116; top: 0; width: 80; height: 12 }
+                    tooltip: "Power"
+                }
             }
         }
-    }
 
-    Form {
-        objectName: "reverse portrait"
-        display: Region { left: 38; top: 60; width: 320; height: 320 }
+        Form {
+            objectName: "reverse portrait"
+            display: Region { left: 38; top: 60; width: 320; height: 320 }
 
-        mainImage: "default_180.png"
-        pressedImage: "default_180_p.png"
+            mainImage: "default_180.png"
+            pressedImage: "default_180_p.png"
 
-        KeyList {
-            Key { objectName: "Power"; keycode: 116
-                Region { left: 0; top: 244; width: 12; height: 80 }
-                tooltip: "Power"
+            KeyList {
+                Key { objectName: "Power"; keycode: 116
+                    Region { left: 0; top: 244; width: 12; height: 80 }
+                    tooltip: "Power"
+                }
             }
         }
-    }
 
-    Form {
-        objectName: "reverse landscape"
-        display: Region { left: 60; top: 26; width: 320; height: 320 }
+        Form {
+            objectName: "reverse landscape"
+            display: Region { left: 60; top: 26; width: 320; height: 320 }
 
-        mainImage: "default_R90.png"
-        pressedImage: "default_R90_p.png"
+            mainImage: "default_R90.png"
+            pressedImage: "default_R90_p.png"
 
-        KeyList {
-            Key { objectName: "Power"; keycode: 116
-                Region { left: 244; top: 372; width: 80; height: 12 }
-                tooltip: "Power"
+            KeyList {
+                Key { objectName: "Power"; keycode: 116
+                    Region { left: 244; top: 372; width: 80; height: 12 }
+                    tooltip: "Power"
+                }
             }
         }
     }
index 6cb887c6c77ec2df4d9f7446c8549a5c15e54a8a..3b999b84487599fbc66b7a4ded91596aedc3a3b1 100644 (file)
@@ -1,8 +1,16 @@
+obj-$(CONFIG_QT) += emulatoruitype.o moc_emulatoruitype.o
+obj-$(CONFIG_QT) += formlisttype.o moc_formlisttype.o
 obj-$(CONFIG_QT) += formtype.o moc_formtype.o
 obj-$(CONFIG_QT) += hardwarekeytype.o moc_hardwarekeytype.o
 obj-$(CONFIG_QT) += regiontype.o moc_regiontype.o
 obj-$(CONFIG_QT) += keylisttype.o moc_keylisttype.o
 
+$(obj)/moc_emulatoruitype.o: $(obj)/moc_emulatoruitype.cpp
+$(obj)/moc_emulatoruitype.cpp: $(obj)/emulatoruitype.h
+       moc $< -o $@
+$(obj)/moc_formlisttype.o: $(obj)/moc_formlisttype.cpp
+$(obj)/moc_formlisttype.cpp: $(obj)/formlisttype.h
+       moc $< -o $@
 $(obj)/moc_formtype.o: $(obj)/moc_formtype.cpp
 $(obj)/moc_formtype.cpp: $(obj)/formtype.h
        moc $< -o $@
diff --git a/tizen/src/ui/xml/emulatoruitype.cpp b/tizen/src/ui/xml/emulatoruitype.cpp
new file mode 100644 (file)
index 0000000..fc615e3
--- /dev/null
@@ -0,0 +1,17 @@
+#include "emulatoruitype.h"
+
+EmulatorUIType::EmulatorUIType(QObject *parent) :
+    QObject(parent)
+{
+    /* do nothing */
+}
+
+FormListType *EmulatorUIType::formListType()
+{
+    return formList;
+}
+
+void EmulatorUIType::setFormListType(FormListType *formList)
+{
+    this->formList = formList;
+}
diff --git a/tizen/src/ui/xml/emulatoruitype.h b/tizen/src/ui/xml/emulatoruitype.h
new file mode 100644 (file)
index 0000000..298c4a8
--- /dev/null
@@ -0,0 +1,25 @@
+#ifndef EMULATORUITYPE_H
+#define EMULATORUITYPE_H
+
+#include <QObject>
+
+#include "formlisttype.h"
+
+class EmulatorUIType : public QObject
+{
+    Q_OBJECT
+    Q_PROPERTY(FormListType *formListType READ formListType WRITE setFormListType)
+    Q_CLASSINFO("DefaultProperty", "formListType")
+
+public:
+    explicit EmulatorUIType(QObject *parent = 0);
+
+    FormListType *formListType();
+
+private:
+    void setFormListType(FormListType *formList);
+
+    FormListType *formList;
+};
+
+#endif // EMULATORUITYPE_H
diff --git a/tizen/src/ui/xml/formlisttype.cpp b/tizen/src/ui/xml/formlisttype.cpp
new file mode 100644 (file)
index 0000000..fc8dcc3
--- /dev/null
@@ -0,0 +1,17 @@
+#include "formlisttype.h"
+
+FormListType::FormListType(QObject *parent) :
+    QObject(parent)
+{
+    /* do nothing */
+}
+
+QQmlListProperty<FormType> FormListType::formList()
+{
+    return QQmlListProperty<FormType>(this, list);
+}
+
+QList<FormType *> *FormListType::getFormList()
+{
+    return &list;
+}
diff --git a/tizen/src/ui/xml/formlisttype.h b/tizen/src/ui/xml/formlisttype.h
new file mode 100644 (file)
index 0000000..52f5d9a
--- /dev/null
@@ -0,0 +1,26 @@
+#ifndef FORMLISTTYPE_H
+#define FORMLISTTYPE_H
+
+#include <QObject>
+#include <QQmlListProperty>
+
+#include "formtype.h"
+
+class FormListType : public QObject
+{
+    Q_OBJECT
+    Q_PROPERTY(QQmlListProperty<FormType> formList READ formList)
+    Q_CLASSINFO("DefaultProperty", "formList")
+
+public:
+    explicit FormListType(QObject *parent = 0);
+
+    QList<FormType *> *getFormList();
+
+private:
+    QQmlListProperty<FormType> formList();
+
+    QList<FormType *> list;
+};
+
+#endif // FORMLISTTYPE_H
index 6f6e50c9ccb2d819cd328ca141bdf9486223fc03..61419e4b52ba433e8cc560c32da77f1d481b319e 100644 (file)
@@ -3,11 +3,17 @@
 FormType::FormType(QObject *parent) :
     QObject(parent), keyList(NULL)
 {
+    /* do nothing */
 }
 
-void FormType::setRegionType(RegionType *regionType)
+RegionType *FormType::displayRegionType()
 {
-    this->regionType = regionType;
+    return displayRegion;
+}
+
+void FormType::setDisplayRegionType(RegionType *displayRegion)
+{
+    this->displayRegion = displayRegion;
 }
 
 QString FormType::mainImageName() const
@@ -40,11 +46,11 @@ void FormType::setKeyListType(KeyListType *keyList)
     this->keyList = keyList;
 }
 
-QRect FormType::displayRegion()
+QRect FormType::getDisplayRegion()
 {
-    if (regionType == NULL) {
+    if (displayRegion == NULL) {
         return QRect(0, 0, 0, 0);
     }
 
-    return regionType->region();
+    return displayRegion->region();
 }
index 662c0a7bce27f5d4a9a5f1982538fbf9014620ea..cf35de13f83eb81acf47b8baf6538a4e12dc2c66 100644 (file)
@@ -9,27 +9,29 @@
 class FormType : public QObject
 {
     Q_OBJECT
-    Q_PROPERTY(RegionType *display WRITE setRegionType)
+    Q_PROPERTY(RegionType *display READ displayRegionType WRITE setDisplayRegionType)
     Q_PROPERTY(QString mainImage READ mainImageName WRITE setMainImageName)
     Q_PROPERTY(QString pressedImage READ pressedImageName WRITE setPressedImageName)
-    Q_PROPERTY(KeyListType *hwKeyList READ keyListType WRITE setKeyListType)
-    Q_CLASSINFO("DefaultProperty", "hwKeyList")
+    Q_PROPERTY(KeyListType *keyListType READ keyListType WRITE setKeyListType)
+    Q_CLASSINFO("DefaultProperty", "keyListType")
 
 public:
     explicit FormType(QObject *parent = 0);
 
-    void setRegionType(RegionType *regionType);
     QString mainImageName() const;
-    void setMainImageName(QString imgFileName);
     QString pressedImageName() const;
-    void setPressedImageName(QString imgFileName);
     KeyListType *keyListType();
-    void setKeyListType(KeyListType *keyList);
 
-    QRect displayRegion();
+    QRect getDisplayRegion();
 
 private:
-    RegionType *regionType;
+    RegionType *displayRegionType();
+    void setDisplayRegionType(RegionType *displayRegion);
+    void setMainImageName(QString imgFileName);
+    void setPressedImageName(QString imgFileName);
+    void setKeyListType(KeyListType *keyList);
+
+    RegionType *displayRegion;
     QString mainImgName;
     QString pressedImgName;
     KeyListType *keyList;
index 975091e1af330526f9efad3ff698712b2c644e0f..e2667fb0ccc0842b044f3e30751dfade5e982eee 100644 (file)
@@ -3,6 +3,7 @@
 HardwareKeyType::HardwareKeyType(QObject *parent) :
     QObject(parent)
 {
+    /* do nothing */
 }
 
 int HardwareKeyType::keycode() const
@@ -25,16 +26,21 @@ void HardwareKeyType::setTooltip(const QString &tooltip)
     tip = tooltip;
 }
 
-void HardwareKeyType::setRegionType(RegionType *regionType)
+RegionType *HardwareKeyType::keyRegionType()
 {
-    this->regionType = regionType;
+    return keyRegion;
+}
+
+void HardwareKeyType::setKeyRegionType(RegionType *keyRegion)
+{
+    this->keyRegion = keyRegion;
 }
 
 QRect HardwareKeyType::region()
 {
-    if (regionType == NULL) {
+    if (keyRegion == NULL) {
         return QRect(0, 0, 0, 0);
     }
 
-    return regionType->region();
+    return keyRegion->region();
 }
index a63acf154e0e7a2b8aae679187cbb6cb542d0167..4e86af00ed374b202a746e41e9941a4b5a3eb978 100644 (file)
@@ -10,24 +10,26 @@ class HardwareKeyType : public QObject
     Q_OBJECT
     Q_PROPERTY(int keycode READ keycode WRITE setKeycode)
     Q_PROPERTY(QString tooltip READ tooltip WRITE setTooltip)
-    Q_PROPERTY(RegionType *keyRegion WRITE setRegionType)
-    Q_CLASSINFO("DefaultProperty", "keyRegion")
+    Q_PROPERTY(RegionType *keyRegionType READ keyRegionType WRITE setKeyRegionType)
+    Q_CLASSINFO("DefaultProperty", "keyRegionType")
 
 public:
     explicit HardwareKeyType(QObject *parent = 0);
 
     int keycode() const;
-    void setKeycode(int keycode);
     QString tooltip() const;
-    void setTooltip(const QString &tooltip);
-    void setRegionType(RegionType *regionType);
 
     QRect region();
 
 private:
+    void setKeycode(int keycode);
+    void setTooltip(const QString &tooltip);
+    RegionType *keyRegionType();
+    void setKeyRegionType(RegionType *regionType);
+
     int code;
     QString tip;
-    RegionType *regionType;
+    RegionType *keyRegion;
 };
 
 #endif // HARDWAREKEYTYPE_H
index c3b0a7fb50ce6ecd91b53b5c5d495575d93fe396..503ebeef32d7262d68f81c8734c4ea38bd0c549c 100644 (file)
@@ -3,9 +3,16 @@
 KeyListType::KeyListType(QObject *parent) :
     QObject(parent)
 {
+    /* do nothing */
 }
 
-QQmlListProperty<HardwareKeyType> KeyListType::keylist()
+QQmlListProperty<HardwareKeyType> KeyListType::keyListType()
 {
     return QQmlListProperty<HardwareKeyType>(this, list);
 }
+
+QList<HardwareKeyType *> *KeyListType::getKeyList()
+{
+    return &list;
+}
+
index b7f4d3c567eb194a03e1d26e145760726d6064c1..9729fcf0b8fb961e132448ce60ca42dac28739fe 100644 (file)
@@ -4,18 +4,21 @@
 #include <QObject>
 #include <QQmlListProperty>
 
-#include "xml/hardwarekeytype.h"
+#include "hardwarekeytype.h"
 
 class KeyListType : public QObject
 {
     Q_OBJECT
-    Q_PROPERTY(QQmlListProperty<HardwareKeyType> keylist READ keylist)
-    Q_CLASSINFO("DefaultProperty", "keylist")
+    Q_PROPERTY(QQmlListProperty<HardwareKeyType> keyListType READ keyListType)
+    Q_CLASSINFO("DefaultProperty", "keyListType")
 
 public:
     explicit KeyListType(QObject *parent = 0);
 
-    QQmlListProperty<HardwareKeyType> keylist();
+    QList<HardwareKeyType *> *getKeyList();
+
+private:
+    QQmlListProperty<HardwareKeyType> keyListType();
 
     QList<HardwareKeyType *> list;
 };
index ebdded8bd87ac3912b00ead44513ed8a2c63f3b6..d2aaff3186a9a92c9bb4641f48b880ae34b20540 100644 (file)
@@ -3,6 +3,12 @@
 RegionType::RegionType(QObject *parent) :
     QObject(parent), x(0), y(0), w(0), h(0)
 {
+    /* do nothing */
+}
+
+int RegionType::left()
+{
+    return x;
 }
 
 void RegionType::setLeft(int left)
@@ -10,16 +16,31 @@ void RegionType::setLeft(int left)
     x = left;
 }
 
+int RegionType::top()
+{
+    return y;
+}
+
 void RegionType::setTop(int top)
 {
     y = top;
 }
 
+int RegionType::width()
+{
+    return w;
+}
+
 void RegionType::setWidth(int width)
 {
     w = width;
 }
 
+int RegionType::height()
+{
+    return h;
+}
+
 void RegionType::setHeight(int height)
 {
     h = height;
index 6fb946f1893b070489e477f41a08b47c6e17e604..c6a53a28f4946b4e21b8db0b9b81599cefd72e25 100644 (file)
@@ -7,22 +7,27 @@
 class RegionType : public QObject
 {
     Q_OBJECT
-    Q_PROPERTY(int left WRITE setLeft)
-    Q_PROPERTY(int top WRITE setTop)
-    Q_PROPERTY(int width WRITE setWidth)
-    Q_PROPERTY(int height WRITE setHeight)
+    Q_PROPERTY(int left READ left WRITE setLeft)
+    Q_PROPERTY(int top READ top WRITE setTop)
+    Q_PROPERTY(int width READ width WRITE setWidth)
+    Q_PROPERTY(int height READ height WRITE setHeight)
 
 public:
     explicit RegionType(QObject *parent = 0);
 
+    int left();
+    int top();
+    int width();
+    int height();
+
+    QRect region() const;
+
+private:
     void setLeft(int left);
     void setTop(int top);
     void setWidth(int width);
     void setHeight(int height);
 
-    QRect region() const;
-
-private:
     int x;
     int y;
     int w;