Skip to content

Commit

Permalink
fix: visualization by message doesn't work on shared folder
Browse files Browse the repository at this point in the history
refs: IRIS-4479
  • Loading branch information
mgarbin committed Jul 4, 2023
1 parent db4eb1e commit d30901d
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@ const DragImageContainer = styled.div`
const DragItems: FC<{
messages: IncompleteMessage[];
draggedIds: Record<string, boolean>;
}> = ({ messages, draggedIds }) => {
folderId: string;
}> = ({ messages, draggedIds, folderId }) => {
const items = reduce<typeof draggedIds, MessageListItemProps['item'][]>(
draggedIds,
(acc, v, k) => {
Expand All @@ -51,6 +52,7 @@ const DragItems: FC<{
selecting={false}
visible={false}
deselectAll={noop}
currentFolderId={folderId}
/>
))}
</>
Expand Down Expand Up @@ -194,7 +196,7 @@ export const MessageListComponent: FC<MessageListComponentProps> = memo(
</Container>
)}
<DragImageContainer ref={dragImageRef}>
<DragItems messages={messages} draggedIds={draggedIds ?? {}} />
<DragItems messages={messages} draggedIds={draggedIds ?? {}} folderId={folderId} />
</DragImageContainer>
</>
) : (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ export type ListItemComponentProps = {
deselectAll: () => void;
visible: boolean;
setDraggedIds?: (ids: Record<string, boolean>) => void;
currentFolderId?: string;
};

export const MessageListItemComponent: FC<ListItemComponentProps> = memo(
Expand All @@ -36,7 +37,8 @@ export const MessageListItemComponent: FC<ListItemComponentProps> = memo(
isSearchModule,
deselectAll,
visible,
setDraggedIds = noop
setDraggedIds = noop,
currentFolderId
}) {
return (
<DragItemWrapper
Expand All @@ -57,6 +59,7 @@ export const MessageListItemComponent: FC<ListItemComponentProps> = memo(
visible={visible}
isSearchModule={isSearchModule}
deselectAll={deselectAll}
currentFolderId={currentFolderId}
/>
</DragItemWrapper>
);
Expand Down
1 change: 1 addition & 0 deletions src/views/app/folder-panel/messages/message-list-item.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ export const MessageListItem: FC<MessageListItemProps> = memo(function MessageLi
currentFolderId
}) {
const firstChildFolderId = currentFolderId ?? item.parent;

const dispatch = useAppDispatch();
const zimbraPrefMarkMsgRead = useUserSettings()?.prefs?.zimbraPrefMarkMsgRead !== '-1';

Expand Down
3 changes: 2 additions & 1 deletion src/views/app/folder-panel/messages/message-list.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,7 @@ export const MessageList: FC = () => {
deselectAll={deselectAll}
visible={visible}
setDraggedIds={setDraggedIds}
currentFolderId={folderId}
/>
) : (
<div style={{ height: '4rem' }} />
Expand All @@ -121,7 +122,7 @@ export const MessageList: FC = () => {
</CustomListItem>
);
}),
[deselectAll, draggedIds, isSelectModeOn, itemId, messages, selected, toggle]
[deselectAll, draggedIds, isSelectModeOn, itemId, messages, selected, toggle, folderId]
);

const totalMessages = useMemo(
Expand Down

0 comments on commit d30901d

Please sign in to comment.