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

Fix icon picker issues on mobile #7113

Merged
merged 9 commits into from
Jan 3, 2025
Merged

Conversation

asjqkkkk
Copy link
Collaborator

@asjqkkkk asjqkkkk commented Dec 31, 2024

Fix issue #7101

  • fix: error displaying in Page style
  • fix: complete the filter logic of icon picker
  • fix: error displaying in Favorite/Recent page
  • fix: the color picker showed when tapping down
  • fix: icons are not supported in subpage blocks

Feature Preview


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.

@asjqkkkk asjqkkkk changed the title Fix #7101 Fix icon picker issues on mobile Dec 31, 2024
@LucasXu0
Copy link
Collaborator

@asjqkkkk We need to write tests to cover these two bug fixes.

  • fix: icons are not supported in subpage blocks
  • fix: error displaying in Page style

@asjqkkkk
Copy link
Collaborator Author

@asjqkkkk We need to write tests to cover these two bug fixes.

  • fix: icons are not supported in subpage blocks
  • fix: error displaying in Page style

All tests have been added

@Xazin Xazin mentioned this pull request Jan 1, 2025
4 tasks
@LucasXu0 LucasXu0 merged commit 15deb8e into AppFlowy-IO:main Jan 3, 2025
19 checks passed
asjqkkkk added a commit to asjqkkkk/AppFlowy that referenced this pull request Jan 3, 2025
* fix: error displaying in Page style

* fix: error displaying in Favorite/Recent page

* fix: complete the filter logic of icon picker

* fix: the color picker showed when tapping down

* fix: icons are not supported in subpage blocks

* chore: add some tests

* fix: recent icons not working for grid header icon
@asjqkkkk asjqkkkk deleted the fix/icons branch January 3, 2025 03:34
asjqkkkk added a commit to asjqkkkk/AppFlowy that referenced this pull request Jan 3, 2025
* fix: error displaying in Page style

* fix: error displaying in Favorite/Recent page

* fix: complete the filter logic of icon picker

* fix: the color picker showed when tapping down

* fix: icons are not supported in subpage blocks

* chore: add some tests

* fix: recent icons not working for grid header icon
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants