Merge lp:~cellsoftware/telegram-app/1530698 into lp:telegram-app

Proposed by Paz Chauhan
Status: Merged
Approved by: Jin
Approved revision: 178
Merged at revision: 178
Proposed branch: lp:~cellsoftware/telegram-app/1530698
Merge into: lp:telegram-app
Diff against target: 37 lines (+5/-1)
2 files modified
telegram/app/qml/AccountMessageAction.qml (+5/-0)
telegram/app/qml/components/MessagesListItem.qml (+0/-1)
To merge this branch: bzr merge lp:~cellsoftware/telegram-app/1530698
Reviewer Review Type Date Requested Status
Jin (community) Approve
Ash Approve
Review via email: mp+292817@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Ash (ashley-richards) wrote :

I have overviewed Paz's QML code and, to me, it all looks fine.

Revision history for this message
Ash (ashley-richards) :
review: Approve
178. By Paz Chauhan

.po and .pot files reverted

Revision history for this message
Jin (jindallo) wrote :

The code looks good to me,
and verified pass locally.

Thanks for this patch,
I will top-approve.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'telegram/app/qml/AccountMessageAction.qml'
2--- telegram/app/qml/AccountMessageAction.qml 2016-04-17 01:29:55 +0000
3+++ telegram/app/qml/AccountMessageAction.qml 2016-04-26 11:27:11 +0000
4@@ -2,6 +2,7 @@
5 import Ubuntu.Components 1.3
6 import AsemanTools 1.0
7 import TelegramQML 1.0
8+import QtQuick.Window 2.0
9
10 Item {
11 id: message_action
12@@ -41,9 +42,13 @@
13 visible: hasAction
14
15 Label {
16+ id: actionText
17 anchors.horizontalCenter: parent.horizontalCenter
18+ width: Screen.width * 0.85
19+ horizontalAlignment: Text.AlignHCenter
20 fontSize: "small"
21 color: "#333333"
22+ wrapMode: Text.WrapAtWordBoundaryOrAnywhere
23 text: {
24 var res = ""
25 var userName
26
27=== modified file 'telegram/app/qml/components/MessagesListItem.qml'
28--- telegram/app/qml/components/MessagesListItem.qml 2016-04-06 08:01:54 +0000
29+++ telegram/app/qml/components/MessagesListItem.qml 2016-04-26 11:27:11 +0000
30@@ -230,7 +230,6 @@
31
32 Label {
33 id: message_text
34-
35 // Taken from messaging-app
36 function parseText(text) {
37 var phoneExp = /(\+?([0-9]+[ ]?)?\(?([0-9]+)\)?[-. ]?([0-9]+)[-. ]?([0-9]+)[-. ]?([0-9]+))/img;

Subscribers

People subscribed via source and target branches

to status/vote changes: