[PR #5443] [MERGED] fix: launch review 0.5.9 #6816

New issue

Closed

opened 2026-03-23 23:16:48 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:16:48 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/5443
Author: @Xazin
Created: 5/30/2024
Status: ✅ Merged
Merged: 6/2/2024
Merged by: @Xazin

Base: main ← Head: fix/launch-review-0.5.9

📝 Commits (10+)

ba9a5ff fix: lose focus in editor on open settings dialog
3e2d9db fix: support CTRL+. for sidebar toggle
56be7bf fix: make notify method private
4e9e476 fix: copy for video block
f79b06f fix: copy for notification setting
73ee43d fix: add libmpv to appimage builder
d7f5a4b chore: merge branch 'upstream/main' into fix/launch-review-0.5.9
ff1c3ef fix: missing tabs bloc from context
4c97146 ci: add libmpv-dev to missing workflows
5f6912d fix: do not depend on inherited widget in dispose

📊 Changes

15 files changed (+221 additions, -111 deletions)

View changed files

📝 .github/actions/flutter_build/action.yml (+1 -1)
📝 .github/actions/flutter_integration_test/action.yml (+1 -1)
📝 .github/workflows/release.yml (+2 -2)
📝 frontend/appflowy_flutter/integration_test/shared/base.dart (+3 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart (+32 -8)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/video/upload_video_menu.dart (+10 -1)
📝 frontend/appflowy_flutter/lib/shared/patterns/common_patterns.dart (+5 -0)
➕ frontend/appflowy_flutter/lib/workspace/presentation/home/af_focus_manager.dart (+40 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/desktop_home_screen.dart (+61 -54)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/hotkeys.dart (+17 -5)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/favorites/favorite_more_actions.dart (+4 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/shared/sidebar_setting.dart (+35 -31)
📝 frontend/appflowy_flutter/linux/flutter/dart_ffi/binding.h (+3 -1)
📝 frontend/resources/translations/en.json (+6 -5)
📝 frontend/scripts/linux_distribution/appimage/AppImageBuilder.yml (+1 -0)

📄 Description

Relates: #4970 #5434

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/5443 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 5/30/2024 **Status:** ✅ Merged **Merged:** 6/2/2024 **Merged by:** [@Xazin](https://github.com/Xazin) **Base:** `main` ← **Head:** `fix/launch-review-0.5.9` --- ### 📝 Commits (10+) - [`ba9a5ff`](https://github.com/AppFlowy-IO/AppFlowy/commit/ba9a5ffa95c186a0b0552b76c5270f53245b53f9) fix: lose focus in editor on open settings dialog - [`3e2d9db`](https://github.com/AppFlowy-IO/AppFlowy/commit/3e2d9dbae6dd367db106a07229483b52f8d0fed5) fix: support CTRL+. for sidebar toggle - [`56be7bf`](https://github.com/AppFlowy-IO/AppFlowy/commit/56be7bf3ddc4b86ece249e8dda17b11456b0b68c) fix: make notify method private - [`4e9e476`](https://github.com/AppFlowy-IO/AppFlowy/commit/4e9e476d538f6327e23c86b9b58e3b7e3404530d) fix: copy for video block - [`f79b06f`](https://github.com/AppFlowy-IO/AppFlowy/commit/f79b06f88154513581e2a068dffe41c300400c6e) fix: copy for notification setting - [`73ee43d`](https://github.com/AppFlowy-IO/AppFlowy/commit/73ee43d1d6744518faba7e111cb1b2862a585398) fix: add libmpv to appimage builder - [`d7f5a4b`](https://github.com/AppFlowy-IO/AppFlowy/commit/d7f5a4b167f3533782ce390dee454b32ce36bb95) chore: merge branch 'upstream/main' into fix/launch-review-0.5.9 - [`ff1c3ef`](https://github.com/AppFlowy-IO/AppFlowy/commit/ff1c3ef9143719cfbdc70acac5b999557a7f6c97) fix: missing tabs bloc from context - [`4c97146`](https://github.com/AppFlowy-IO/AppFlowy/commit/4c97146aca7a8bb5adf8bfc80913d1bb6226c1cc) ci: add libmpv-dev to missing workflows - [`5f6912d`](https://github.com/AppFlowy-IO/AppFlowy/commit/5f6912da43bbd8b0f963fe2ccf1925c0bb396e89) fix: do not depend on inherited widget in dispose ### 📊 Changes **15 files changed** (+221 additions, -111 deletions) <details> <summary>View changed files</summary> 📝 `.github/actions/flutter_build/action.yml` (+1 -1) 📝 `.github/actions/flutter_integration_test/action.yml` (+1 -1) 📝 `.github/workflows/release.yml` (+2 -2) 📝 `frontend/appflowy_flutter/integration_test/shared/base.dart` (+3 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart` (+32 -8) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/video/upload_video_menu.dart` (+10 -1) 📝 `frontend/appflowy_flutter/lib/shared/patterns/common_patterns.dart` (+5 -0) ➕ `frontend/appflowy_flutter/lib/workspace/presentation/home/af_focus_manager.dart` (+40 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/desktop_home_screen.dart` (+61 -54) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/hotkeys.dart` (+17 -5) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/favorites/favorite_more_actions.dart` (+4 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/shared/sidebar_setting.dart` (+35 -31) 📝 `frontend/appflowy_flutter/linux/flutter/dart_ffi/binding.h` (+3 -1) 📝 `frontend/resources/translations/en.json` (+6 -5) 📝 `frontend/scripts/linux_distribution/appimage/AppImageBuilder.yml` (+1 -0) </details> ### 📄 Description Relates: #4970 #5434 #### PR Checklist - [x] 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>