Page Menu
Home
Phorge
Search
Configure Global Search
Log In
Files
F32236464
D7630.1765222213.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Flag For Later
Award Token
Size
7 KB
Referenced Files
None
Subscribers
None
D7630.1765222213.diff
View Options
diff --git a/native/chat/chat-input-bar.react.js b/native/chat/chat-input-bar.react.js
--- a/native/chat/chat-input-bar.react.js
+++ b/native/chat/chat-input-bar.react.js
@@ -84,6 +84,7 @@
import { ChatContext } from './chat-context.js';
import type { ChatNavigationProp } from './chat.react.js';
+import type { RemoveEditMode } from './message-list-types.js';
import TypeaheadTooltip from './typeahead-tooltip.react.js';
import Button from '../components/button.react.js';
// eslint-disable-next-line import/extensions
@@ -169,13 +170,14 @@
text: string,
) => Promise<SendEditMessageResponse>,
+navigation: ?ChatNavigationProp<'MessageList'>,
+ +isFocused?: boolean,
};
type State = {
+text: string,
+textEdited: boolean,
+buttonsExpanded: boolean,
+selectionState: SyncedSelectionData,
- +isExitingEditMode: boolean,
+ +isExitingDuringEditMode: boolean,
};
class ChatInputBar extends React.PureComponent<Props, State> {
textInput: ?React.ElementRef<typeof TextInput>;
@@ -202,7 +204,7 @@
textEdited: false,
buttonsExpanded: true,
selectionState: { text: props.draft, selection: { start: 0, end: 0 } },
- isExitingEditMode: false,
+ isExitingDuringEditMode: false,
};
this.setUpActionIconAnimations();
@@ -408,6 +410,22 @@
this.expandButtons();
this.setIOSKeyboardHeight();
}
+
+ if (!this.props.isFocused && prevProps.isFocused && this.isEditMode()) {
+ this.setState(
+ { text: this.props.draft, isExitingDuringEditMode: true },
+ this.exitEditMode,
+ );
+ }
+ if (this.props.isFocused && !prevProps.isFocused) {
+ this.setState({ isExitingDuringEditMode: false });
+ }
+ if (
+ this.props.inputState?.editState.editedMessage &&
+ !prevProps.inputState?.editState.editedMessage
+ ) {
+ this.blockNavigation();
+ }
}
addEditInputMessageListener() {
@@ -733,9 +751,12 @@
};
updateText = (text: string) => {
+ if (this.state.isExitingDuringEditMode) {
+ return;
+ }
this.setState({ text, textEdited: true });
this.props.inputState?.setEditedMessageChanged(this.isMessageEdited(text));
- if (this.isEditMode() || this.state.isExitingEditMode) {
+ if (this.isEditMode()) {
return;
}
this.saveDraft(text);
@@ -851,6 +872,41 @@
return text !== originalText;
};
+ unblockNavigation = () => {
+ if (!this.props.navigation) {
+ return;
+ }
+ this.props.navigation.setParams({ removeEditMode: null });
+ };
+
+ removeEditMode: RemoveEditMode = action => {
+ if (!this.props.navigation) {
+ return;
+ }
+ if (this.state.isExitingDuringEditMode) {
+ return;
+ }
+ const nav = this.props.navigation;
+ const unblockAndDispatch = () => {
+ this.unblockNavigation();
+ nav.dispatch(action);
+ };
+ if (!this.isMessageEdited()) {
+ unblockAndDispatch();
+ return;
+ }
+ exitEditAlert(unblockAndDispatch);
+ };
+
+ blockNavigation = () => {
+ if (!this.props.navigation) {
+ return;
+ }
+ this.props.navigation.setParams({
+ removeEditMode: this.removeEditMode,
+ });
+ };
+
editMessage = async (messageID: string, text: string) => {
if (!this.isMessageEdited()) {
this.exitEditMode();
@@ -896,6 +952,7 @@
exitEditMode = () => {
this.props.inputState?.setEditedMessage(null, () => {
+ this.unblockNavigation();
this.updateText(this.props.draft);
this.focusAndUpdateButtonsVisibility();
this.updateSendButton(this.props.draft);
@@ -910,7 +967,7 @@
e.preventDefault();
const saveExit = () => {
this.props.inputState?.setEditedMessage(null, () => {
- this.setState({ isExitingEditMode: true }, () => {
+ this.setState({ isExitingDuringEditMode: true }, () => {
if (!this.props.navigation) {
return;
}
@@ -1107,6 +1164,7 @@
+onInputBarLayout?: (event: LayoutEvent) => mixed,
+openCamera: () => mixed,
+navigation?: ChatNavigationProp<'MessageList'>,
+ +isFocused?: boolean,
};
function ConnectedChatInputBarBase(props: ConnectedChatInputBarBaseProps) {
const navContext = React.useContext(NavContext);
@@ -1297,12 +1355,15 @@
});
}, [keyboardState, navigation, route.key, threadInfo]);
+ const isFocused = props.navigation.isFocused();
+
return (
<ConnectedChatInputBarBase
{...restProps}
onInputBarLayout={onInputBarLayout}
openCamera={openCamera}
navigation={navigation}
+ isFocused={isFocused}
/>
);
});
diff --git a/native/chat/chat-router.js b/native/chat/chat-router.js
--- a/native/chat/chat-router.js
+++ b/native/chat/chat-router.js
@@ -20,6 +20,7 @@
clearThreadsActionType,
pushNewThreadActionType,
} from '../navigation/action-types.js';
+import { getRemoveEditMode } from '../navigation/nav-selectors.js';
import {
removeScreensFromStack,
getThreadIDFromRoute,
@@ -128,7 +129,18 @@
);
return baseGetStateForAction(clearedState, navigateAction, options);
} else {
- return baseGetStateForAction(lastState, action, options);
+ const result = baseGetStateForAction(lastState, action, options);
+ const removeEditMode = getRemoveEditMode(lastState);
+ if (
+ result !== null &&
+ result?.index &&
+ result.index > lastState.index &&
+ removeEditMode
+ ) {
+ removeEditMode(action);
+ return lastState;
+ }
+ return result;
}
},
actionCreators: {
diff --git a/native/chat/message-list-types.js b/native/chat/message-list-types.js
--- a/native/chat/message-list-types.js
+++ b/native/chat/message-list-types.js
@@ -7,6 +7,7 @@
import type { ThreadInfo } from 'lib/types/thread-types.js';
import { type UserInfo } from 'lib/types/user-types.js';
+import type { ChatRouterNavigationAction } from './chat-router.js';
import type { MarkdownRules } from '../markdown/rules.react.js';
import { useTextMessageRulesFunc } from '../markdown/rules.react.js';
import { MessageListRouteName } from '../navigation/route-names.js';
@@ -15,8 +16,11 @@
+threadInfo: ThreadInfo,
+pendingPersonalThreadUserInfo?: UserInfo,
+searching?: boolean,
+ +removeEditMode?: ?RemoveEditMode,
};
+export type RemoveEditMode = (action: ChatRouterNavigationAction) => void;
+
export type MessageListContextType = {
+getTextMessageMarkdownRules: (useDarkStyle: boolean) => MarkdownRules,
};
diff --git a/native/navigation/nav-selectors.js b/native/navigation/nav-selectors.js
--- a/native/navigation/nav-selectors.js
+++ b/native/navigation/nav-selectors.js
@@ -31,6 +31,7 @@
threadRoutes,
CommunityDrawerNavigatorRouteName,
} from './route-names.js';
+import type { RemoveEditMode } from '../chat/message-list-types';
import { useSelector } from '../redux/redux-utils.js';
import type { NavPlusRedux } from '../types/selector-types.js';
import type { GlobalTheme } from '../types/themes.js';
@@ -316,6 +317,26 @@
},
);
+const getRemoveEditMode: (
+ navigationState: ?PossiblyStaleNavigationState,
+) => ?RemoveEditMode = (
+ currentTabSubrouteState: ?PossiblyStaleNavigationState,
+) => {
+ if (!currentTabSubrouteState) {
+ return null;
+ }
+ const currentChatRoute =
+ currentTabSubrouteState.routes[currentTabSubrouteState.routes.length - 1];
+ if (currentChatRoute.name !== MessageListRouteName) {
+ return null;
+ }
+ if (!currentChatRoute || !currentChatRoute.params) {
+ return null;
+ }
+ const removeEditMode: Function = currentChatRoute.params.removeEditMode;
+ return removeEditMode;
+};
+
function useCurrentLeafRouteName(): ?string {
const navContext = React.useContext(NavContext);
return React.useMemo(() => {
@@ -342,4 +363,5 @@
useCalendarQuery,
drawerSwipeEnabledSelector,
useCurrentLeafRouteName,
+ getRemoveEditMode,
};
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Mon, Dec 8, 7:30 PM (7 h, 35 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
5850170
Default Alt Text
D7630.1765222213.diff (7 KB)
Attached To
Mode
D7630: [native] Blocking navigation in edit mode in ChatRouter
Attached
Detach File
Event Timeline
Log In to Comment