tristan/kdeconnect-kde
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Don't show phonenumber if name is available

Browse source 
Summary:
Showing the number next to the name is neither pretty nor necessary IMHO.

My Android SMS app displays the number more subtle in an extra line below in gray. I could get behind something similar, but until this is implemented just don't show it

Depends on D16015

Reviewers: #kde_connect, sredman

Reviewed By: #kde_connect, sredman

Subscribers: sredman, kdeconnect

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D16017
This commit is contained in:
Nicolas Fella 2018-10-08 20:37:16 +02:00
parent 45e966c290
commit 8bfe093939
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
smsapp/qml/ConversationDisplay.qml
View file

@ -36,7 +36,7 @@ Kirigami.ScrollablePage
property string conversationId
property string phoneNumber
title: person.person && person.person.name ? i18n("%1: %2", person.person.name, phoneNumber) : phoneNumber
title: person.person && person.person.name ? person.person.name : phoneNumber
ListView {
model: QSortFilterProxyModel {