From 54e8f8e600bf54b0e73c95590748a79bfaf65acf Mon Sep 17 00:00:00 2001 From: Dominic Go <18517029+dominicstop@users.noreply.github.com> Date: Fri, 7 Apr 2023 03:44:58 +0800 Subject: [PATCH] =?UTF-8?q?=F0=9F=92=84=20Gloss:=20Update=20Comments?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../React Native/RNIModal/RNIModalManager.swift | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/ios/src_library/React Native/RNIModal/RNIModalManager.swift b/ios/src_library/React Native/RNIModal/RNIModalManager.swift index 26d56323..78dcdf90 100644 --- a/ios/src_library/React Native/RNIModal/RNIModalManager.swift +++ b/ios/src_library/React Native/RNIModal/RNIModalManager.swift @@ -9,7 +9,18 @@ import Foundation public let RNIModalManagerShared = RNIModalManager.sharedInstance; -/// Note:2023-03-30-19-36-33 +/// `Note:2023-04-07-03-22-48` +/// +/// * Manually incrementing and decrementing the `modalIndex` is fragile. +/// +/// * For example: +/// * 1 -If multiple blur/focus events were to fire consecutively, the +/// `modalIndex` might be wrong. +/// +/// * 2 - If a modal presented/dismissed w/o notifying `RNIModalManager`, +/// the `modalIndex` will be stale. +/// +/// `Note:2023-03-30-19-36-33` - Archived/Old /// /// * This assumes that the app is using a single window, and all the modals are /// being presented on that window, as a chain of view controllers.