-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Prevent format emoji On Submitted text #40617
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
Prevent format emoji On Submitted text #40617
Conversation
…x-14676-prevent-format-emoji
…x-14676-prevent-format-emoji
…x-14676-prevent-format-emoji
…b.com/wildan-m/App into wildan/fix-14676-prevent-format-emoji
So I actually think we'd want to have the emoji's ignore the formatting in both preview and when it's sent. I believe our general philosophy when it comes to preview is that it should generally be one-to-one with what gets sent. And from the screenshot it looks like the emoji's are italic in the preview, but not when it's sent: Keen on @Expensify/design input here as well |
@dubielzyk-expensify unformat the emoji in preview is possible, but according to this. That might be handled separately because it's using a live markdown library maintained by an agency. @eh2077 is that correct? |
I see, if that's the case, then I'm fine with it. Thanks for clarifying 😄 |
@shawnborton Yes, we should fix composer as well and it'll be fixed in the other repo react-native-live-markdown which is a project supported by SWM team. We'll ask them to fix it in the main GH issue once this PR is merged. Does this sound good to you? |
Cool, that works for me - thanks for explaining! |
@wildan-m Friendly bump! In case you missed this #40617 (comment) |
…x-14676-prevent-format-emoji
…x-14676-prevent-format-emoji
@dubielzyk-expensify @shawnborton What is the expected hover behavior for emojis inside InlineCodeBlock? Should we display a tooltip? |
Hmm I don't feel too strongly about that. Is it possible to continue supporting the tooltip when it's wrapped in a code block? Otherwise I think it's totally fine if we don't have one there. |
Yeah, I'm okay with not having one. Don't feel very strongly either though |
Reviewer Checklist
Screenshots/Videos |
Looks good and tested well. @wildan-m Please help to mark comments as resolved. |
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.
Looks awesome! So glad this is finally fixed!
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging by https://github.com/stitesExpensify in version: 1.4.69-0 🚀
|
🚀 Deployed to production by https://github.com/Beamanator in version: 1.4.69-2 🚀
|
@eh2077
Details
Feature request: Prevent submitted emoji text from being formatted as italic, bold, or strikethrough.
Fixed Issues
$ #14676
PROPOSAL: #14676 (comment)
Tests
Markdown 1
Markdown 2
Offline tests
Same as Tests
QA Steps
Same as Tests
PR Author Checklist
### Fixed Issues
section aboveTests
sectionOffline steps
sectionQA steps
sectiontoggleReport
and notonIconClick
)myBool && <MyComponent />
.src/languages/*
files and using the translation methodSTYLE.md
) were followedAvatar
, I verified the components usingAvatar
are working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)
)Avatar
is modified, I verified thatAvatar
is working as expected in all cases)Design
label and/or tagged@Expensify/design
so the design team can review the changes.ScrollView
component to make it scrollable when more elements are added to the page.main
branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTest
steps.Screenshots/Videos
Android: Native
Kapture.2024-04-20.at.10.32.00.mp4
Android: mWeb Chrome
Kapture.2024-04-20.at.09.33.22.mp4
iOS: Native
Kapture.2024-04-20.at.10.18.04.mp4
iOS: mWeb Safari
Kapture.2024-04-20.at.09.23.19.mp4
MacOS: Chrome / Safari
Kapture.2024-04-20.at.09.15.52.mp4
MacOS: Desktop
Kapture.2024-04-20.at.10.23.58.mp4