Merge lp:~tiagosh/messaging-app/show_notification_mic_icon into lp:messaging-app

[Tiago Salem Herrmann]

Show notification when the the record icon is only clicked.

Show notification when the the record icon is only clicked.

[lp:~tiagosh/messaging-app/show_notification_mic_icon]

517. By Tiago Salem Herrmann on 2016-02-25

Show notification when the the record icon is only clicked

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:517
http://jenkins.qa.ubuntu.com/job/messaging-app-ci/964/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6650/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/messaging-app-vivid-i386-ci/473/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6661/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/messaging-app-ci/964/rebuild

review: Needs Fixing (continuous-integration)

[lp:~tiagosh/messaging-app/show_notification_mic_icon]

518. By Tiago Salem Herrmann on 2016-03-03

Use popover instead of notification

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:518
http://jenkins.qa.ubuntu.com/job/messaging-app-ci/967/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6700/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/messaging-app-vivid-i386-ci/476/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6711/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/messaging-app-ci/967/rebuild

review: Needs Fixing (continuous-integration)

[lp:~tiagosh/messaging-app/show_notification_mic_icon]

519. By Tiago Salem Herrmann on 2016-03-04

Avoid dismissing the osk

520. By Tiago Salem Herrmann on 2016-03-04

Close popup when text changes

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:520
http://jenkins.qa.ubuntu.com/job/messaging-app-ci/970/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6711/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/messaging-app-vivid-i386-ci/479/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6722/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/messaging-app-ci/970/rebuild

review: Needs Fixing (continuous-integration)

[Gustavo Pichorim Boiko (boiko)]

Can you make the background of the popup dark? (maybe follow the translucent black scheme used by the notifications?)?
The contrast between the hint and the rest of the content on screen is really bad, which makes it hard to notice.

review: Needs Fixing

[Gustavo Pichorim Boiko (boiko)]

Looks good!

review: Approve

[lp:~tiagosh/messaging-app/show_notification_mic_icon]

521. By Tiago Salem Herrmann on 2016-03-17

Fix osk focus on new message screen

=== modified file 'src/qml/ComposeBar.qml'
--- src/qml/ComposeBar.qml	2016-03-07 21:16:20 +0000
+++ src/qml/ComposeBar.qml	2016-03-17 18:41:13 +0000
@@ -138,6 +138,32 @@
         }
     }
 
+    Component {
+        id: microphoneWarningPopover
+
+        Popover {
+            id: popover
+            Column {
+                id: containerLayout
+                anchors {
+                    left: parent.left
+                    top: parent.top
+                    right: parent.right
+                }
+                ListItem.Standard {
+                    text: i18n.tr("You have to press and hold the record icon")
+                    onClicked: {
+                        PopupUtils.close(popover)
+                    }
+                }
+                Connections {
+                    target: composeBar
+                    onTextChanged: PopupUtils.close(popover)
+                }
+            }
+        }
+    }
+
     ListItem.ThinDivider {
         anchors.top: parent.top
     }
@@ -551,6 +577,21 @@
         iconColor: composeBar.recording ? "black" : "gray"
         iconName: "audio-input-microphone-symbolic"
 
+        onClicked: {
+            if (!composeBar.audioAttached) {
+                var oskFocus = null
+                if (textEntry.activeFocus) {
+                    oskFocus = composeBar.forceFocus
+                } else if (multiRecipient.activeFocus) {
+                    oskFocus = multiRecipient.forceActiveFocus
+                }
+                PopupUtils.open(microphoneWarningPopover, recordButton)
+                // avoid dismissing the osk
+                if (oskFocus) {
+                    oskFocus()
+                }
+            }
+        }
         onPressed: audioRecordingBar.startRecording()
         onReleased: {
             audioRecordingBar.stopRecording()