mirror of
https://github.com/AppFlowy-IO/AppFlowy.git
synced 2026-03-24 04:46:56 +00:00
[PR #5531] [MERGED] Fix space issues #6865
Labels
No labels
2024
2025
2026
acct mgmt
AI
automation
bug
calendar
ci
CJK
cloud
code-block
collaboration
copy-paste
database
data migration
data sync
deploy
desktop
develop
develop
documentation
duplicate
editor
editor-plugin
emoji
export
files
flutter-only
follow-up
formula
good first issue for devs
good first issue for experienced devs
grid
hacktoberfest
HACKTOBERFEST-ACCEPTED
help wanted
i18n
icons
images
importer
improvements
infra
install
integrations
IR
kanban board
login
look and joy
mentorship
mobile
mobile
needs design
new feature
new feature
non-coding
notes
notifications
onboarding
organization
P0+
permission
platform-linux
platform-mac
platform-windows
plugins
program
pull-request
Q1 25
Q1 26
Q2 24
Q2 25
Q3 24
Q3 25
Q4 24
Q4 25
react
regression
rust
rust
Rust-only
Rust-only
Rust-starter
Rust-starter
self-hosted
shortcuts
side panel
slash-menu
sync v2
table
tablet
task
tauri
templates
tests
themes
translation
v0.5.6
v0.5.8
v0.5.9
v0.6.0
v0.6.1
v0.6.4
v0.6.7
v0.6.8
v0.7.1
v0.7.4
v0.7.4
v0.7.5
v0.7.6
v0.7.7
v0.7.8
v0.8.0
v0.8.4
v0.8.5
v0.8.9
web
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
AppFlowy-IO/AppFlowy#6865
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 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:
main← Head:fix_space_issues📝 Commits (10+)
2b3c239fix: display the space icon in breadcrumb and disable the space entryfa138a4fix: the icon and the title are not aligned center in more menu410d800fix: incorrect space icon corner radiusf5bc53bfix: missed tooltip for add new page button691b4eafix: disable space migration for local user74945dachore: use general as default space nameb8bfe1cfix: space name overflowe8b649dfix: only show arrow button when hovering on page31624d0fix: sidebar tooltip font sizee572f26fix: 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

The icon and the title are not aligned center.

incorrect space icon corner radius

missed tooltip

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
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.