Photoviewer example fixes.

2 files changed, 16 insertions, 28 deletions

diff --git a/demos/declarative/photoviewer/PhotoViewerCore/Button.qml b/demos/declarative/photoviewer/PhotoViewerCore/Button.qml
index c6810646eb..5be096a322 100644
--- a/demos/declarative/photoviewer/PhotoViewerCore/Button.qml
+++ b/demos/declarative/photoviewer/PhotoViewerCore/Button.qml
@@ -4,7 +4,7 @@ Item {
     id: container
 
     property alias label: labelText.text
-    property color tint: "#FFFFFFFF"
+    property string tint: ""
     signal clicked
 
     width: labelText.width + 70 ; height: labelText.height + 18
@@ -19,7 +19,7 @@ Item {
 
     Rectangle {
         anchors.fill: container; color: container.tint; visible: container.tint != ""
-        opacity: 0.1; smooth: true
+        opacity: 0.25; smooth: true
     }
 
     Text { id: labelText; font.pixelSize: 15; anchors.centerIn: parent; smooth: true }
diff --git a/demos/declarative/photoviewer/PhotoViewerCore/EditableButton.qml b/demos/declarative/photoviewer/PhotoViewerCore/EditableButton.qml
index ccfda02526..15ffe567fc 100644
--- a/demos/declarative/photoviewer/PhotoViewerCore/EditableButton.qml
+++ b/demos/declarative/photoviewer/PhotoViewerCore/EditableButton.qml
@@ -4,11 +4,10 @@ Item {
     id: container
 
     property string label
-    property color tint: "#FFFFFFFF"
     signal clicked
     signal labelChanged(string label)
 
-    width: labelText.width + 70 ; height: labelText.height + 18
+    width: textInput.width + 70 ; height: textInput.height + 18
 
     BorderImage {
         anchors { fill: container; leftMargin: -6; topMargin: -6; rightMargin: -8; bottomMargin: -8 }
@@ -18,36 +17,25 @@ Item {
 
     Image { anchors.fill: parent; source: "images/cardboard.png"; smooth: true }
 
-    Rectangle {
-        anchors.fill: container; color: container.tint; visible: container.tint != ""
-        opacity: 0.1; smooth: true
-    }
-
-    Text { id: labelText; text: label; font.pixelSize: 15; anchors.centerIn: parent; smooth: true }
-
     TextInput {
-        id: textInput; text: label; font.pixelSize: 15; anchors.centerIn: parent; smooth: true; visible: false
-        Keys.onReturnPressed: container.labelChanged(textInput.text)
+        id: textInput; text: label; font.pixelSize: 15; anchors.centerIn: parent; smooth: true
+        Keys.onReturnPressed: {
+            container.labelChanged(textInput.text)
+            container.focus = true
+        }
         Keys.onEscapePressed: {
-            textInput.text = labelText.text
-            container.state = ''
+            textInput.text = container.label
+            container.focus = true
         }
     }
 
-    MouseArea {
-        anchors { fill: parent; leftMargin: -20; topMargin: -20; rightMargin: -20; bottomMargin: -20 }
-        onClicked: container.state = "editMode"
-    }
-
-    states: State {
-        name: "editMode"
-        PropertyChanges { target: container; width: textInput.width + 70; height: textInput.height + 17 }
-        PropertyChanges { target: textInput; visible: true; focus: true }
-        PropertyChanges { target: labelText; visible: false }
+    Rectangle {
+        anchors.fill: container; border.color: "steelblue"; border.width: 4
+        color: "transparent"; visible: textInput.focus; smooth: true
     }
 
-    onLabelChanged: {
-        labelText.text = label
-        container.state = ''
+    MouseArea {
+        anchors { fill: parent; leftMargin: -20; topMargin: -20; rightMargin: -20; bottomMargin: -20 }
+        onClicked: textInput.forceFocus()
     }
 }