Skip to content

Commit

Permalink
fix image save tooltip, fix missing save in info panel (keybase#25608)
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisnojima authored Apr 6, 2023
1 parent aa97d76 commit c89679a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
7 changes: 4 additions & 3 deletions shared/chat/conversation/messages/attachment/image2/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import * as Kb from '../../../../../common-adapters'
import * as React from 'react'
import * as Styles from '../../../../../styles'
import ImageImpl from './imageimpl'
import {Title, useAttachmentRedux, useCollapseIcon, Collapsed} from '../shared'
import {ShowToastAfterSaving, Title, useAttachmentRedux, useCollapseIcon, Collapsed} from '../shared'

type Props = {
toggleMessageMenu: () => void
Expand All @@ -11,7 +11,7 @@ type Props = {

const Image2 = React.memo(function Image2(p: Props) {
const {isHighlighted, toggleMessageMenu} = p
const {fileName, isCollapsed, isEditing, showTitle, openFullscreen} = useAttachmentRedux()
const {fileName, isCollapsed, isEditing, showTitle, openFullscreen, transferState} = useAttachmentRedux()
const containerStyle = isHighlighted || isEditing ? styles.containerHighlighted : styles.container
const collapseIcon = useCollapseIcon(false)

Expand All @@ -34,6 +34,7 @@ const Image2 = React.memo(function Image2(p: Props) {
alignSelf="flex-start"
alignItems="flex-start"
>
<ShowToastAfterSaving transferState={transferState} />
<Kb.ClickableBox
onClick={openFullscreen}
onLongPress={toggleMessageMenu}
Expand All @@ -45,7 +46,7 @@ const Image2 = React.memo(function Image2(p: Props) {
</Kb.Box2>
</>
)
}, [filename, openFullscreen, toggleMessageMenu, showTitle])
}, [filename, openFullscreen, toggleMessageMenu, showTitle, transferState])

return (
<Kb.Box2 direction="vertical" fullWidth={true} style={containerStyle} alignItems="flex-start">
Expand Down
1 change: 0 additions & 1 deletion shared/common-adapters/save-indicator.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,6 @@ class SaveIndicator extends React.Component<Props, State> {
}

render() {
return null // TEMP
return (
<Kb.Box style={Styles.collapseStyles([defaultStyle, this.props.style] as any)}>
{this.getChildren()}
Expand Down

0 comments on commit c89679a

Please sign in to comment.