[PR #5263] [MERGED] fix: launch review improvements #6737

Closed
opened 2026-03-23 23:16:26 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/5263
Author: @Xazin
Created: 5/5/2024
Status: Merged
Merged: 5/6/2024
Merged by: @LucasXu0

Base: mainHead: fix/launch-review


📝 Commits (8)

  • ecb2ccd fix: recent views fix
  • db01473 fix: text button font color
  • c73c278 chore: merge branch 'upstream/main' into fix/launch-review
  • 76fb80e fix: sidebar scrollbar inset
  • fe75865 fix: account settings launch review
  • 3565688 fix: open + menu on mobile provider issue
  • 63109bf fix: code review
  • ec291c7 fix: push view improvement

📊 Changes

27 files changed (+271 additions, -365 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/mobile/application/mobile_router.dart (+2 -0)
📝 frontend/appflowy_flutter/lib/mobile/presentation/favorite/mobile_favorite_folder.dart (+5 -12)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/favorite_folder/mobile_home_favorite_folder.dart (+3 -4)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/recent_folder/mobile_home_recent_views.dart (+9 -9)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/section_folder/mobile_home_section_folder.dart (+3 -4)
📝 frontend/appflowy_flutter/lib/mobile/presentation/page_item/mobile_view_item.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/setting/setting_button.dart (+3 -7)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart (+11 -15)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/base/emoji_picker_button.dart (+25 -32)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/code_block/code_language_screen.dart (+3 -5)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/font/customize_font_toolbar_item.dart (+4 -4)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mobile_toolbar_v3/add_block_toolbar_item.dart (+22 -34)
📝 frontend/appflowy_flutter/lib/workspace/application/recent/cached_recent_service.dart (+5 -2)
📝 frontend/appflowy_flutter/lib/workspace/application/recent/recent_views_bloc.dart (+7 -3)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/home_stack.dart (+6 -5)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/folder/_favorite_folder.dart (+6 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/folder/_folder_header.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar.dart (+4 -8)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar_folder.dart (+28 -39)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_item.dart (+2 -1)

...and 7 more files

📄 Description

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.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/AppFlowy-IO/AppFlowy/pull/5263 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 5/5/2024 **Status:** ✅ Merged **Merged:** 5/6/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix/launch-review` --- ### 📝 Commits (8) - [`ecb2ccd`](https://github.com/AppFlowy-IO/AppFlowy/commit/ecb2ccda72d34da29cdad309531b280b207a94fd) fix: recent views fix - [`db01473`](https://github.com/AppFlowy-IO/AppFlowy/commit/db01473887ab0e1db8917f2daece0c4e99c9316d) fix: text button font color - [`c73c278`](https://github.com/AppFlowy-IO/AppFlowy/commit/c73c278a861cb7ee96035b3c14859296f5e3b44e) chore: merge branch 'upstream/main' into fix/launch-review - [`76fb80e`](https://github.com/AppFlowy-IO/AppFlowy/commit/76fb80e2ee38590218744abc55be37fecead0a96) fix: sidebar scrollbar inset - [`fe75865`](https://github.com/AppFlowy-IO/AppFlowy/commit/fe758654b59f1a5fa351ed7a1248024d90a52bae) fix: account settings launch review - [`3565688`](https://github.com/AppFlowy-IO/AppFlowy/commit/35656887b64af679c374722207690b89d636f2dc) fix: open + menu on mobile provider issue - [`63109bf`](https://github.com/AppFlowy-IO/AppFlowy/commit/63109bfcff179a02a82a7d4105170bb4c22a534b) fix: code review - [`ec291c7`](https://github.com/AppFlowy-IO/AppFlowy/commit/ec291c760eb626d2d019b1e2b5dd4742b5a10164) fix: push view improvement ### 📊 Changes **27 files changed** (+271 additions, -365 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/mobile/application/mobile_router.dart` (+2 -0) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/favorite/mobile_favorite_folder.dart` (+5 -12) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/favorite_folder/mobile_home_favorite_folder.dart` (+3 -4) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/recent_folder/mobile_home_recent_views.dart` (+9 -9) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/section_folder/mobile_home_section_folder.dart` (+3 -4) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/page_item/mobile_view_item.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/setting/setting_button.dart` (+3 -7) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart` (+11 -15) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/base/emoji_picker_button.dart` (+25 -32) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/code_block/code_language_screen.dart` (+3 -5) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/font/customize_font_toolbar_item.dart` (+4 -4) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mobile_toolbar_v3/add_block_toolbar_item.dart` (+22 -34) 📝 `frontend/appflowy_flutter/lib/workspace/application/recent/cached_recent_service.dart` (+5 -2) 📝 `frontend/appflowy_flutter/lib/workspace/application/recent/recent_views_bloc.dart` (+7 -3) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/home_stack.dart` (+6 -5) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/folder/_favorite_folder.dart` (+6 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/folder/_folder_header.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar.dart` (+4 -8) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar_folder.dart` (+28 -39) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_item.dart` (+2 -1) _...and 7 more files_ </details> ### 📄 Description ### Feature Preview <!--- List at least one issue here that this PR addresses. If it fixes the issue, please use the [fixes](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests) keyword to close the issue. For example: fixes https://github.com/AppFlowy-IO/AppFlowy/pull/2106 --> --- <!--- Before you mark this PR ready for review, run through this checklist! --> #### PR Checklist - [ ] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/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. - [x] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
mirror 2026-03-23 23:16:26 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#6737
No description provided.