Skip to content
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: support recent section in icon picker #7015

Merged
merged 5 commits into from
Dec 26, 2024

Conversation

asjqkkkk
Copy link
Collaborator

@asjqkkkk asjqkkkk commented Dec 19, 2024

Feature Preview

Add the recent icons and emojis to the selector

Recent Emoji Recent Icon
image image

Bug fixes

  • Make the Tooltip corresponding to Emojis appear above in the selector.
  • Dismiss Tooltip while scrolling
  • Improve the logic to filter emojis or icons by keyword ([Bug] Icon search missing result #6248)
  • The focus disappears after the Emoji Picker is opened

[Filter Emoji] Before:

emoji_search_before.mp4

[Filter Emoji] After:

emoji_search_after.mp4

PR Checklist

  • My code adheres to AppFlowy's Conventions
  • I've listed at least one issue that this PR fixes in the description above.
  • I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
  • All existing tests are passing.

@LucasXu0 LucasXu0 changed the title Feat/icons feat: support recent section in icon picker Dec 20, 2024
@asjqkkkk asjqkkkk force-pushed the feat/icons branch 3 times, most recently from 2a695f8 to 6c3a454 Compare December 23, 2024 07:57
Co-authored-by: Mathias Mogensen <[email protected]>
Co-authored-by: LucasXu0 <[email protected]>
@asjqkkkk asjqkkkk force-pushed the feat/icons branch 4 times, most recently from ecb2d8a to 7422ea2 Compare December 25, 2024 04:35
@LucasXu0 LucasXu0 merged commit e67e9cc into AppFlowy-IO:main Dec 26, 2024
19 checks passed
@asjqkkkk asjqkkkk deleted the feat/icons branch December 26, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants