Long tap on pie chart let the user cancels the media download.

matrixConsole/matrixConsole/Base.lproj/Main.storyboard

@@ -99,8 +99,8 @@
                                                 <view hidden="YES" contentMode="scaleToFill" translatesAutoresizingMaskIntoConstraints="NO" id="yUK-od-5YZ" userLabel="ProgressView">
                                                     <rect key="frame" x="487" y="-1" width="100" height="70"/>
                                                     <subviews>
-                                                        <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" misplaced="YES" text="rate" textAlignment="center" lineBreakMode="tailTruncation" numberOfLines="3" baselineAdjustment="alignBaselines" minimumFontSize="4" preferredMaxLayoutWidth="100" translatesAutoresizingMaskIntoConstraints="NO" id="HFo-GV-TO9" userLabel="Progress stats">
-                                                            <rect key="frame" x="0.0" y="59" width="100" height="11"/>
+                                                        <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="rate" textAlignment="center" lineBreakMode="tailTruncation" numberOfLines="3" baselineAdjustment="alignBaselines" minimumFontSize="4" preferredMaxLayoutWidth="100" translatesAutoresizingMaskIntoConstraints="NO" id="HFo-GV-TO9" userLabel="Progress stats">
+                                                            <rect key="frame" x="0.0" y="60" width="100" height="10"/>
                                                             <fontDescription key="fontDescription" type="system" pointSize="8"/>
                                                             <color key="textColor" cocoaTouchSystemColor="darkTextColor"/>
                                                             <nil key="highlightedColor"/>
@@ -113,6 +113,7 @@
                                                             </constraints>
                                                         </view>
                                                     </subviews>
+                                                    <gestureRecognizers/>
                                                     <constraints>
                                                         <constraint firstItem="HFo-GV-TO9" firstAttribute="leading" secondItem="yUK-od-5YZ" secondAttribute="leading" id="2Xg-Bu-iAg"/>
                                                         <constraint firstAttribute="centerX" secondItem="pg4-aQ-7qW" secondAttribute="centerX" id="FQt-WU-G5b"/>
@@ -123,6 +124,9 @@
                                                         <constraint firstItem="pg4-aQ-7qW" firstAttribute="top" secondItem="yUK-od-5YZ" secondAttribute="top" id="mEw-ej-VSR"/>
                                                         <constraint firstAttribute="trailing" secondItem="HFo-GV-TO9" secondAttribute="trailing" id="rF7-vW-wgF"/>
                                                     </constraints>
+                                                    <connections>
+                                                        <outletCollection property="gestureRecognizers" destination="TmC-iG-UPj" appends="YES" id="MmM-Jr-ywe"/>
+                                                    </connections>
                                                 </view>
                                             </subviews>
                                             <color key="backgroundColor" white="0.0" alpha="0.0" colorSpace="calibratedWhite"/>
@@ -202,8 +206,8 @@
                                                 <view hidden="YES" contentMode="scaleToFill" translatesAutoresizingMaskIntoConstraints="NO" id="own-aM-Zlf" userLabel="ProgressView">
                                                     <rect key="frame" x="18" y="-1" width="100" height="70"/>
                                                     <subviews>
-                                                        <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" misplaced="YES" text="text" textAlignment="center" lineBreakMode="tailTruncation" numberOfLines="3" baselineAdjustment="alignBaselines" minimumFontSize="4" preferredMaxLayoutWidth="100" translatesAutoresizingMaskIntoConstraints="NO" id="L87-yV-XGk" userLabel="Progress stats">
-                                                            <rect key="frame" x="0.0" y="59" width="100" height="11"/>
+                                                        <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="text" textAlignment="center" lineBreakMode="tailTruncation" numberOfLines="3" baselineAdjustment="alignBaselines" minimumFontSize="4" preferredMaxLayoutWidth="100" translatesAutoresizingMaskIntoConstraints="NO" id="L87-yV-XGk" userLabel="Progress stats">
+                                                            <rect key="frame" x="0.0" y="60" width="100" height="10"/>
                                                             <fontDescription key="fontDescription" type="system" pointSize="8"/>
                                                             <color key="textColor" cocoaTouchSystemColor="darkTextColor"/>
                                                             <nil key="highlightedColor"/>
@@ -216,6 +220,7 @@
                                                             </constraints>
                                                         </view>
                                                     </subviews>
+                                                    <gestureRecognizers/>
                                                     <constraints>
                                                         <constraint firstItem="L87-yV-XGk" firstAttribute="leading" secondItem="own-aM-Zlf" secondAttribute="leading" id="BFY-SA-2gW"/>
                                                         <constraint firstAttribute="height" constant="70" id="Hg0-st-iq2"/>
@@ -226,6 +231,9 @@
                                                         <constraint firstAttribute="centerX" secondItem="L87-yV-XGk" secondAttribute="centerX" id="vMM-Hu-UkI"/>
                                                         <constraint firstAttribute="centerX" secondItem="lTG-14-OWN" secondAttribute="centerX" id="weT-lI-eKE"/>
                                                     </constraints>
+                                                    <connections>
+                                                        <outletCollection property="gestureRecognizers" destination="bv4-jS-RnJ" appends="YES" id="ZPg-Ay-HeL"/>
+                                                    </connections>
                                                 </view>
                                                 <imageView clipsSubviews="YES" userInteractionEnabled="NO" contentMode="scaleAspectFill" horizontalHuggingPriority="251" verticalHuggingPriority="251" image="default-profile.png" translatesAutoresizingMaskIntoConstraints="NO" id="mks-jh-AiZ" customClass="CustomImageView">
                                                     <rect key="frame" x="552" y="5" width="40" height="40"/>
@@ -511,6 +519,16 @@
                     </connections>
                 </viewController>
                 <placeholder placeholderIdentifier="IBFirstResponder" id="94y-cU-qQD" userLabel="First Responder" sceneMemberID="firstResponder"/>
+                <pongPressGestureRecognizer allowableMovement="10" minimumPressDuration="0.5" id="TmC-iG-UPj">
+                    <connections>
+                        <action selector="onProgressLongTap:" destination="msb-ol-2LB" id="no2-Q3-7ya"/>
+                    </connections>
+                </pongPressGestureRecognizer>
+                <pongPressGestureRecognizer allowableMovement="10" minimumPressDuration="0.5" id="bv4-jS-RnJ">
+                    <connections>
+                        <action selector="onProgressLongTap:" destination="msb-ol-2LB" id="8ks-us-qqh"/>
+                    </connections>
+                </pongPressGestureRecognizer>
             </objects>
             <point key="canvasLocation" x="1595" y="75"/>
         </scene>

matrixConsole/matrixConsole/View/RoomMessageTableCell.h

@@ -42,6 +42,7 @@
 - (void)startProgressUI;
 - (void)stopProgressUI;
 
+- (void)cancelDownload;
 @end
 
 @interface IncomingMessageTableCell : RoomMessageTableCell

matrixConsole/matrixConsole/View/RoomMessageTableCell.m

@@ -21,6 +21,11 @@
 
 @implementation RoomMessageTableCell
 
+- (void)dealloc {
+    // remove any pending observers
+    [[NSNotificationCenter defaultCenter] removeObserver:self];
+}
+
 - (void)updateProgressUI:(NSDictionary*)downloadStatsDict {
     
     self.progressView.hidden = NO;
@@ -70,6 +75,12 @@
         
         if ([url isEqualToString:self.message.attachmentURL]) {
             [self stopProgressUI];
+            
+            // the job is really over
+            if ([notif.name isEqualToString:kMediaDownloadDidFinishNotification]) {
+                // remove any pending observers
+                [[NSNotificationCenter defaultCenter] removeObserver:self];
+            }
         }
     }
 }
@@ -78,6 +89,9 @@
     
     BOOL isHidden = YES;
     
+    // remove any pending observers
+    [[NSNotificationCenter defaultCenter] removeObserver:self];
+    
     // there is an attachment URL
     if (self.message.attachmentURL) {
         
@@ -105,10 +119,20 @@
 - (void)stopProgressUI {
     self.progressView.hidden = YES;
     
-    // remove the observers
-    [[NSNotificationCenter defaultCenter] removeObserver:self name:kMediaDownloadProgressNotification object:nil];
-    [[NSNotificationCenter defaultCenter] removeObserver:self name:kMediaDownloadDidFinishNotification object:nil];
-    [[NSNotificationCenter defaultCenter] removeObserver:self name:kMediaDownloadDidFailNotification object:nil];
+    // do not remove the observer here
+    // the download could restart without recomposing the cell
+}
+
+- (void)cancelDownload {
+    // get the linked medida loader
+    id loader = [MediaManager mediaLoaderForURL:self.message.attachmentURL];
+    
+    if (loader) {
+        [MediaManager cancel:loader];
+    }
+    
+    // ensure there is no more progress bar
+    [self stopProgressUI];
 }
 @end
 

matrixConsole/matrixConsole/ViewController/RoomViewController.m

@@ -329,6 +329,34 @@ NSString *const kCmdResetUserPowerLevel = @"/deop";
     return YES;
 }
 
+- (IBAction)onProgressLongTap:(UILongPressGestureRecognizer*)sender {
+    if (sender.state == UIGestureRecognizerStateBegan) {
+        // find out the linked uitableviewcell
+        UIView* view = sender.view;
+        
+        while(view && ![view isKindOfClass:[UITableViewCell class]]) {
+            view = view.superview;
+        }
+        
+        // if it is a RoomMessageTableCell
+        if ([view isKindOfClass:[RoomMessageTableCell class]]) {
+            __weak typeof(self) weakSelf = self;
+            
+            
+            self.actionMenu = [[CustomAlert alloc] initWithTitle:nil message:@"Cancel the download ?" style:CustomAlertStyleAlert];
+            self.actionMenu.cancelButtonIndex = [self.actionMenu addActionWithTitle:@"Cancel" style:CustomAlertActionStyleDefault handler:^(CustomAlert *alert) {
+                weakSelf.actionMenu = nil;
+            }];
+            self.actionMenu.cancelButtonIndex = [self.actionMenu addActionWithTitle:@"OK" style:CustomAlertActionStyleDefault handler:^(CustomAlert *alert) {
+               [(RoomMessageTableCell*)view cancelDownload];
+                weakSelf.actionMenu = nil;
+            }];
+            
+            [self.actionMenu showInViewController:self];
+        }
+    }
+}
+
 #pragma mark - Internal methods
 
 - (void)configureView {