Merge branch 'Deselect-after-Segue' of https://github.com/funnel20/InAppSettingsKit

Author: futuretap

Sources/InAppSettingsKit/Controllers/IASKAppSettingsViewController.m

@@ -902,13 +902,38 @@ - (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath
 
 		NSString *segueIdentifier = specifier.segueIdentifier;
         if (segueIdentifier) {
-			@try {
-				[self performSegueWithIdentifier:segueIdentifier sender:self];
-			} @catch (NSException *exception) {
-				NSLog(@"segue with identifier '%@' not defined", segueIdentifier);
-				[tableView deselectRowAtIndexPath:indexPath animated:YES];
-			}
-			[tableView endUpdates];
+            @try {
+                [self performSegueWithIdentifier:segueIdentifier
+                                          sender:self];
+                
+                /*
+                 NOTE: The segue's viewcontroller will be presented modally. Hence any delegate methods on this class, like `viewWillDisappear:` and `viewWillAppear:`, will not be called when that viewcontroller is presented and dismissed. Consequently the selected row is not deselected, since that will be handled in `viewWillAppear:`.
+                 To resolve this, a repeating timer is used to check whether the presented view controller gets dismissed.
+                 */
+                NSTimer *checkSegueDismissedTimer = [NSTimer timerWithTimeInterval:0.1
+                                                                           repeats:YES
+                                                                             block:^(NSTimer * _Nonnull timer) {
+                    // When the presented view controller is dismissed:
+                    if (self.presentedViewController == nil) {
+                        // Deselect row:
+                        [self.tableView deselectRowAtIndexPath:indexPath
+                                                      animated:YES];
+                        
+                        // Stop timer:
+                        if ([timer isValid]) {
+                            [timer invalidate];
+                        }
+                    }
+                }];
+                
+                // Add timer to the Main runloop:
+                [[NSRunLoop mainRunLoop] addTimer:checkSegueDismissedTimer
+                                          forMode:NSRunLoopCommonModes];
+            } @catch (NSException *exception) {
+                NSLog(@"segue with identifier '%@' not defined", segueIdentifier);
+                [tableView deselectRowAtIndexPath:indexPath animated:YES];
+            }
+            [tableView endUpdates];
             return;
         }