Verified Commit 4e565cab authored by Linus Jahn's avatar Linus Jahn

ChatMessage: Rename isRead property to isDelivered

Fixes #309.
parent 28986ab6
Pipeline #5782 passed with stages
in 20 minutes and 32 seconds
......@@ -181,7 +181,7 @@ Kirigami.ScrollablePage {
sentByMe: model.sentByMe
messageBody: model.body
dateTime: new Date(model.timestamp)
isRead: model.isDelivered
isDelivered: model.isDelivered
name: chatName
mediaType: model.mediaType
mediaGetUrl: model.mediaUrl
......@@ -43,7 +43,7 @@ RowLayout {
property bool sentByMe: true
property string messageBody
property date dateTime
property bool isRead: false
property bool isDelivered: false
property int mediaType
property string mediaGetUrl
property string mediaLocation
......@@ -249,7 +249,7 @@ RowLayout {
// message meta: date, isRead
// message meta: date, isDelivered
RowLayout {
// progress bar for upload/download status
Controls.ProgressBar {
......@@ -267,7 +267,7 @@ RowLayout {
Image {
id: checkmark
visible: (sentByMe && isRead)
  • I think we should make those changes instead:

    visible: sentByMe enabled: isDekivered

    So that the check mark is always visible for our sent messages, and it be disabled until it is delivered. For me it make more sense. what do you think ?

Please register or sign in to reply
visible: (sentByMe && isDelivered)
source: kaidan.utils.getResourcePath("images/message_checkmark.svg")
Layout.preferredHeight: Kirigami.Units.gridUnit * 0.65
Layout.preferredWidth: Kirigami.Units.gridUnit * 0.65
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment