-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: display language icon for auto translated msgs #5398
feat: display language icon for auto translated msgs #5398
Conversation
@diegolmello I've fixed the issue sir. Could you please take a look when you have a chance? Let me know if there are any further changes needed. Thanks. |
import { themes } from '../../../../lib/constants'; | ||
import styles from '../../styles'; | ||
|
||
const Translated = memo(({ isTranslated }: { isTranslated: boolean }) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modified: app/containers/message/Message.stories.tsx
@reinaldonetof Sir can you review this pr when you are free, Thank you |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's almost there 🚀
import styles from '../../styles'; | ||
|
||
const Translated = memo(({ isTranslated }: { isTranslated: boolean }) => { | ||
const { theme } = useTheme(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const { theme } = useTheme(); | |
const { colors } = useTheme(); |
|
||
return ( | ||
<View style={styles.rightIcons}> | ||
<CustomIcon name='language' size={16} color={themes[theme].auxiliaryText} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<CustomIcon name='language' size={16} color={themes[theme].auxiliaryText} /> | |
<CustomIcon name='language' size={16} color={colors.auxiliaryText} /> |
@reinaldonetof I've made the requested changes sir. Please check it out when you have time, Thanks |
@GleidsonDaniel Can you review this pr in your free time? Thanks |
Put it before the read receipt |
e5f6670
to
92d27b6
Compare
outdated
Proposed changes
When the message is auto translated, renders the ‘language’ icon on the Message’s RightIcons
Issue(s)
closes #4807
How to test or reproduce
Screenshots
Types of changes
Checklist
Further comments