[PR #5531] [MERGED] Fix space issues #6865

Closed
opened 2026-03-23 23:17:02 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/5531
Author: @LucasXu0
Created: 6/13/2024
Status: Merged
Merged: 6/14/2024
Merged by: @LucasXu0

Base: mainHead: fix_space_issues


📝 Commits (10+)

  • 2b3c239 fix: display the space icon in breadcrumb and disable the space entry
  • fa138a4 fix: the icon and the title are not aligned center in more menu
  • 410d800 fix: incorrect space icon corner radius
  • f5bc53b fix: missed tooltip for add new page button
  • 691b4ea fix: disable space migration for local user
  • 74945da chore: use general as default space name
  • b8bfe1c fix: space name overflow
  • e8b649d fix: only show arrow button when hovering on page
  • 31624d0 fix: sidebar tooltip font size
  • e572f26 fix: use space name as hint text

📊 Changes

16 files changed (+149 additions, -57 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/workspace/application/favorite/favorite_service.dart (+13 -1)
📝 frontend/appflowy_flutter/lib/workspace/application/recent/cached_recent_service.dart (+15 -2)
📝 frontend/appflowy_flutter/lib/workspace/application/sidebar/space/space_bloc.dart (+11 -2)
📝 frontend/appflowy_flutter/lib/workspace/application/view_title/view_title_bloc.dart (+10 -3)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/header/sidebar_top_menu.dart (+5 -3)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar.dart (+7 -3)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/create_space_popup.dart (+4 -4)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/manage_space_popup.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/sidebar_space.dart (+32 -17)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/sidebar_space_header.dart (+13 -8)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/space_icon_popup.dart (+3 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/space_more_popup.dart (+4 -4)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/navigation.dart (+4 -4)
📝 frontend/appflowy_flutter/lib/workspace/presentation/widgets/view_title_bar.dart (+20 -1)
📝 frontend/resources/translations/en.json (+3 -2)
📝 frontend/rust-lib/flowy-folder/src/manager.rs (+4 -2)

📄 Description

Feature Preview

closes https://github.com/AppFlowy-IO/AppFlowy/issues/5530

  • disable the space migration for local user

  • display the space icon in breadcrumb and disable the space entry
    Screenshot 2024-06-13 at 15 11 36

  • The icon and the title are not aligned center.
    Screenshot 2024-06-13 at 15 13 54

  • incorrect space icon corner radius
    Screenshot 2024-06-13 at 15 15 01

  • missed tooltip
    Screenshot 2024-06-13 at 15 16 38

  • use general as default space name

  • space name overflows if it's too long

  • only show the expand icon when hovering on the pages

  • support adding a new space from space more menu

  • filter the space view from favorites and recent views


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/5531 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 6/13/2024 **Status:** ✅ Merged **Merged:** 6/14/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix_space_issues` --- ### 📝 Commits (10+) - [`2b3c239`](https://github.com/AppFlowy-IO/AppFlowy/commit/2b3c239b8d65be588e8422aaf6f2e37a40bdd971) fix: display the space icon in breadcrumb and disable the space entry - [`fa138a4`](https://github.com/AppFlowy-IO/AppFlowy/commit/fa138a4ecef6d3db63e7eb3a17481595e779c128) fix: the icon and the title are not aligned center in more menu - [`410d800`](https://github.com/AppFlowy-IO/AppFlowy/commit/410d80030b5347ccfd0bd9b9cff3754882967f22) fix: incorrect space icon corner radius - [`f5bc53b`](https://github.com/AppFlowy-IO/AppFlowy/commit/f5bc53bc072f9bb99b427d82ae8d969e0519e820) fix: missed tooltip for add new page button - [`691b4ea`](https://github.com/AppFlowy-IO/AppFlowy/commit/691b4eabe24482a1e21224de316f65f60fda450a) fix: disable space migration for local user - [`74945da`](https://github.com/AppFlowy-IO/AppFlowy/commit/74945da38c8011a031593c5891dcc36ff7f2ed99) chore: use general as default space name - [`b8bfe1c`](https://github.com/AppFlowy-IO/AppFlowy/commit/b8bfe1c3eb5614e4f0c01df90b80797712c59b74) fix: space name overflow - [`e8b649d`](https://github.com/AppFlowy-IO/AppFlowy/commit/e8b649d8f86a43c394909f80cafc3e9e37238a05) fix: only show arrow button when hovering on page - [`31624d0`](https://github.com/AppFlowy-IO/AppFlowy/commit/31624d010cdcf10aa78f308dfee1b55c8913d848) fix: sidebar tooltip font size - [`e572f26`](https://github.com/AppFlowy-IO/AppFlowy/commit/e572f2603b35acc6104bf3db548ad7c10350843c) fix: use space name as hint text ### 📊 Changes **16 files changed** (+149 additions, -57 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/workspace/application/favorite/favorite_service.dart` (+13 -1) 📝 `frontend/appflowy_flutter/lib/workspace/application/recent/cached_recent_service.dart` (+15 -2) 📝 `frontend/appflowy_flutter/lib/workspace/application/sidebar/space/space_bloc.dart` (+11 -2) 📝 `frontend/appflowy_flutter/lib/workspace/application/view_title/view_title_bloc.dart` (+10 -3) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/header/sidebar_top_menu.dart` (+5 -3) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/sidebar.dart` (+7 -3) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/create_space_popup.dart` (+4 -4) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/manage_space_popup.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/sidebar_space.dart` (+32 -17) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/sidebar_space_header.dart` (+13 -8) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/space_icon_popup.dart` (+3 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/space_more_popup.dart` (+4 -4) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/navigation.dart` (+4 -4) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/widgets/view_title_bar.dart` (+20 -1) 📝 `frontend/resources/translations/en.json` (+3 -2) 📝 `frontend/rust-lib/flowy-folder/src/manager.rs` (+4 -2) </details> ### 📄 Description <!--- Thank you for submitting a pull request to AppFlowy. The team will dedicate their best efforts to reviewing and approving your pull request. If you have any questions about the project or feedback for us, please join our [Discord](https://discord.gg/wdjWUXXhtw). --> <!--- If your pull request adds a new feature, please drag and drop a video into this section to showcase what you've done! If not, you may delete this section. --> ### Feature Preview closes https://github.com/AppFlowy-IO/AppFlowy/issues/5530 - [x] disable the space migration for local user - [x] display the space icon in breadcrumb and disable the space entry <img width="205" alt="Screenshot 2024-06-13 at 15 11 36" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/fa9f0a0a-8977-48f5-ac85-f34b63aec8bb"> - [x] The icon and the title are not aligned center. <img width="249" alt="Screenshot 2024-06-13 at 15 13 54" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/ef6c1943-2753-4c9e-bf7c-090012d70b5c"> - [x] incorrect space icon corner radius <img width="477" alt="Screenshot 2024-06-13 at 15 15 01" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/7e427614-6b0d-4128-81e3-8e7509040bcc"> - [x] missed tooltip <img width="337" alt="Screenshot 2024-06-13 at 15 16 38" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/1812e3d2-c706-418b-a1d0-2c51522539fd"> - [x] use general as default space name - [x] space name overflows if it's too long - [x] only show the expand icon when hovering on the pages - [x] support adding a new space from space more menu - [x] filter the space view from favorites and recent views <!--- 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 - [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. - [ ] 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:17:02 +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#6865
No description provided.