← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

[Merge] lp:~artmello/ubuntu-calendar-app/ubuntu-calendar-app-fix_1508440 into lp:ubuntu-calendar-app

 

Arthur Mello has proposed merging lp:~artmello/ubuntu-calendar-app/ubuntu-calendar-app-fix_1508440 into lp:ubuntu-calendar-app.

Commit message:
Set event's title with strike-through text when user is not attending it

Requested reviews:
  Ubuntu Calendar Developers (ubuntu-calendar-dev)
Related bugs:
  Bug #1508440 in Ubuntu Calendar App: "Display events marked as not attending clearly"
  https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1508440

For more details, see:
https://code.launchpad.net/~artmello/ubuntu-calendar-app/ubuntu-calendar-app-fix_1508440/+merge/290091

Set event's title with strike-through text when user is not attending it
-- 
Your team Ubuntu Calendar Developers is requested to review the proposed merge of lp:~artmello/ubuntu-calendar-app/ubuntu-calendar-app-fix_1508440 into lp:ubuntu-calendar-app.
=== modified file 'EventBubble.qml'
--- EventBubble.qml	2016-03-22 20:14:49 +0000
+++ EventBubble.qml	2016-03-24 21:40:37 +0000
@@ -38,6 +38,7 @@
     property Flickable flickable;
     property bool isEventBubble: true
     property real minimumHeight: units.gu(4)
+    property bool strikeoutTitle: false
 
     readonly property bool isSingleLine: (infoBubble.height < (minimumHeight * 2))
     readonly property real startTimeInMinutes: event ? CanlendarCanvas.minutesSince(infoBubble.anchorDate, event.startDateTime) : 0.0
@@ -67,6 +68,7 @@
                 if( getOwnersStatus(collection) === EventAttendee.StatusDeclined ) {
                     //if owner of account is not attending event the dim it
                     bg.color = Qt.tint( collection.color, "#aaffffff" );
+                    infoBubble.strikeoutTitle = true;
                 } else {
                     bg.color = collection.color
                 }
@@ -160,6 +162,7 @@
         fontSize: "small"
         color: "White"
         font.bold: true
+        font.strikeout: infoBubble.strikeoutTitle
     }
 
     Drag.active: dragArea.drag.active


Follow ups