Repository navigation
[HOLD https://github.com/Expensify/react-native-live-markdown/pull/175]Web - Composer - Composer is highlighted but there is no Insertion Pointer ( | ) in composer #36172
Description
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: v1.4.39-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
- Open ND
- Navigate between chats
Expected Result:
Insertion pointer should be displayed in composer
Actual Result:
Insertion pointer is not displayed in composer
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
- Android: Native
- Android: mWeb Chrome
- iOS: Native
- iOS: mWeb Safari
- MacOS: Chrome / Safari
- MacOS: Desktop
Screenshots/Videos
Add any screenshot/video evidence
Activity
github-actions commentedon Feb 8, 2024
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
kbecciv commentedon Feb 8, 2024
We think that this bug might be related #vip-vsb
CC @quinthar
melvin-bot commentedon Feb 8, 2024
Triggered auto assignment to @hayata-suenaga (
Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.allgandalf commentedon Feb 8, 2024
ig it's a regression from #35837, it changes the
RNTextInputtoMarkdownTextInputtomekzaw commentedon Feb 8, 2024
There's already an existing PR from @robertKozik for same thing but for Android & iOS, we could extend that:
thienlnam commentedon Feb 8, 2024
Taking this over since it deals with live markdown
thienlnam commentedon Feb 8, 2024
Doesn't look like it needs to be a blocker, but let's get this resolved soon! Will assign @robertKozik since it looks like we can wrap that into the existing PR
EDIT: just kidding, looks like you will have to comment on the issue for me to do that
robertKozik commentedon Feb 9, 2024
Now you should be able to assign 😄 . I'm going to work on extending the animated input fix now
robertKozik commentedon Feb 9, 2024
Looks like reintroducing animated component don't solve it. It's for sure problem with/lack of autofocus prop. CC. @Skalakid
robertKozik commentedon Feb 12, 2024
This PR contaions fix to the RCA of this issue: Expensify/react-native-live-markdown#175 .
melvin-bot commentedon Feb 12, 2024
@thienlnam Whoops! This issue is 2 days overdue. Let's get this updated quick!
[-]Web - Composer - Composer is highlighted but there is no Insertion Pointer ( | ) in composer[/-][+][HOLD https://github.com/Expensify/react-native-live-markdown/pull/175]Web - Composer - Composer is highlighted but there is no Insertion Pointer ( | ) in composer[/+]robertKozik commentedon Feb 19, 2024
Tested locally on release 0.1.9. Issue is no longer reproductible - Expensify/react-native-live-markdown#175 seems to solve this