[PR #1707] [MERGED] feat: save menu appearance #4907

New issue

Closed

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

mirror commented

2026-03-23 22:16:21 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/1707
Author: @abichinger
Created: 1/14/2023
Status: ✅ Merged
Merged: 1/18/2023
Merged by: @appflowy

Base: main ← Head: feat/save-menu-offset

📝 Commits (2)

6989ee7 feat: save menu offset and menu visibility
91e9988 refactor: remove collapsedNotifier

📊 Changes

9 files changed (+129 additions, -122 deletions)

View changed files

📝 frontend/app_flowy/lib/workspace/application/appearance.dart (+23 -3)
📝 frontend/app_flowy/lib/workspace/application/home/home_setting_bloc.dart (+21 -14)
📝 frontend/app_flowy/lib/workspace/presentation/home/home_layout.dart (+2 -2)
📝 frontend/app_flowy/lib/workspace/presentation/home/home_screen.dart (+10 -17)
📝 frontend/app_flowy/lib/workspace/presentation/home/home_stack.dart (+8 -10)
📝 frontend/app_flowy/lib/workspace/presentation/home/hotkeys.dart (+0 -4)
📝 frontend/app_flowy/lib/workspace/presentation/home/menu/menu.dart (+18 -27)
📝 frontend/app_flowy/lib/workspace/presentation/home/navigation.dart (+35 -45)
📝 frontend/rust-lib/flowy-user/src/entities/user_setting.rs (+12 -0)

📄 Description

solves #690

extended AppearanceSettingsPB with

is_menu_collapsed - the current visibility state of the menu
menu_offset - the current offset of the default menu width

_{🔄 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/1707 **Author:** [@abichinger](https://github.com/abichinger) **Created:** 1/14/2023 **Status:** ✅ Merged **Merged:** 1/18/2023 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `feat/save-menu-offset` --- ### 📝 Commits (2) - [`6989ee7`](https://github.com/AppFlowy-IO/AppFlowy/commit/6989ee7bcebe7f102e094f888bc02573ce19e395) feat: save menu offset and menu visibility - [`91e9988`](https://github.com/AppFlowy-IO/AppFlowy/commit/91e998854c017c1562b07b44e62fb5be90e5aafe) refactor: remove collapsedNotifier ### 📊 Changes **9 files changed** (+129 additions, -122 deletions) <details> <summary>View changed files</summary> 📝 `frontend/app_flowy/lib/workspace/application/appearance.dart` (+23 -3) 📝 `frontend/app_flowy/lib/workspace/application/home/home_setting_bloc.dart` (+21 -14) 📝 `frontend/app_flowy/lib/workspace/presentation/home/home_layout.dart` (+2 -2) 📝 `frontend/app_flowy/lib/workspace/presentation/home/home_screen.dart` (+10 -17) 📝 `frontend/app_flowy/lib/workspace/presentation/home/home_stack.dart` (+8 -10) 📝 `frontend/app_flowy/lib/workspace/presentation/home/hotkeys.dart` (+0 -4) 📝 `frontend/app_flowy/lib/workspace/presentation/home/menu/menu.dart` (+18 -27) 📝 `frontend/app_flowy/lib/workspace/presentation/home/navigation.dart` (+35 -45) 📝 `frontend/rust-lib/flowy-user/src/entities/user_setting.rs` (+12 -0) </details> ### 📄 Description solves #690 extended `AppearanceSettingsPB` with - `is_menu_collapsed` - the current visibility state of the menu - `menu_offset` - the current offset of the default menu width --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>