-
Notifications
You must be signed in to change notification settings - Fork 3.9k
phase1: fix emoji picker refocus issue #35572
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
Merged
Merged
Changes from all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
1457b8b
fix emoji picker keyboard issue
ntdiary a0e50a5
Merge branch 'main' into fix-restore-focus-29011
ntdiary 814a798
Merge branch 'main' into fix-restore-focus-29011
ntdiary 70e763c
fix lint error
ntdiary 6c134cc
polish code
ntdiary 024bd2b
Update src/libs/ComposerFocusManager.ts
ntdiary e85ca86
small improvements
ntdiary 3754145
optional improvement
ntdiary 251e467
function improvement
ntdiary fd73677
upload field improvement
ntdiary 4df50f0
Merge branch 'main' into fix-restore-focus-29011
ntdiary bc482fb
fix lint error
ntdiary 120b3ef
Merge branch 'main' into fix-restore-focus-29011
ntdiary 628c44d
fix switch bug
ntdiary 2371784
code style improvement
ntdiary daf140c
Merge branch 'main' into fix-restore-focus-29011
ntdiary 6fe725d
fix lint error
ntdiary f64a7bc
Merge branch 'main' into fix-restore-focus-29011
ntdiary 6ac3184
Merge branch 'main' into fix-restore-focus-29011
ntdiary d943c4d
Merge branch 'main' into fix-restore-focus-29011
ntdiary d7d5a87
fix selection bug
ntdiary a45d86d
Merge branch 'main' into fix-restore-focus-29011
ntdiary 2a46033
Merge branch 'main' into fix-restore-focus-29011
ntdiary File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| import type {ReactNode} from 'react'; | ||
| import React from 'react'; | ||
|
|
||
| type ModalContentProps = { | ||
| /** Modal contents */ | ||
| children: ReactNode; | ||
|
|
||
| /** | ||
| * Callback method fired after modal content is unmounted. | ||
| * isVisible is not enough to cover all modal close cases, | ||
| * such as closing the attachment modal through the browser's back button. | ||
| * */ | ||
| onDismiss: () => void; | ||
| }; | ||
|
|
||
| function ModalContent({children, onDismiss = () => {}}: ModalContentProps) { | ||
| // eslint-disable-next-line react-hooks/exhaustive-deps | ||
| React.useEffect(() => () => onDismiss?.(), []); | ||
|
ntdiary marked this conversation as resolved.
|
||
| return children; | ||
| } | ||
| ModalContent.displayName = 'ModalContent'; | ||
|
|
||
| export default ModalContent; | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.