[PR #6899] [MERGED] fix: tabs lr 0.7.6 #7626

Closed
opened 2026-03-23 23:20:29 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6899
Author: @Xazin
Created: 12/1/2024
Status: Merged
Merged: 12/1/2024
Merged by: @Xazin

Base: mainHead: fix/tabs-lr-0.7.6


📝 Commits (3)

  • 324aeb4 fix: tabs improvements launch review
  • 7464a8a test: add tests for pin/unpin feature
  • 69b2c99 fix: failing test + cleanup

📊 Changes

15 files changed (+488 additions, -168 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/uncategorized/tabs_test.dart (+136 -39)
📝 frontend/appflowy_flutter/lib/plugins/ai_chat/chat.dart (+6 -1)
📝 frontend/appflowy_flutter/lib/plugins/blank/blank.dart (+4 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/tab_bar/tab_bar_view.dart (+5 -1)
📝 frontend/appflowy_flutter/lib/plugins/database_document/database_document_plugin.dart (+6 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/document.dart (+6 -1)
📝 frontend/appflowy_flutter/lib/plugins/trash/trash.dart (+5 -4)
📝 frontend/appflowy_flutter/lib/workspace/application/tabs/tabs_bloc.dart (+120 -16)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/home_stack.dart (+11 -4)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/workspace/_sidebar_workspace_menu.dart (+7 -8)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/navigation.dart (+5 -5)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/tabs/flowy_tab.dart (+140 -81)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/tabs/tabs_manager.dart (+3 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/widgets/tab_bar_item.dart (+30 -5)
📝 frontend/resources/translations/en.json (+4 -1)

📄 Description

  • Close tab menu popover when it should be closed
  • Make "Close"-option disabled when pinned
  • Add tooltips to buttons in Tab menu to explain usage or when they're disabled (why)
  • Close tabs when switching workspace
  • Move pinned tabs to beginning of tab bar
  • Include View Icon in tab
  • Pinned tabs are smaller and only show icon if possible

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/6899 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 12/1/2024 **Status:** ✅ Merged **Merged:** 12/1/2024 **Merged by:** [@Xazin](https://github.com/Xazin) **Base:** `main` ← **Head:** `fix/tabs-lr-0.7.6` --- ### 📝 Commits (3) - [`324aeb4`](https://github.com/AppFlowy-IO/AppFlowy/commit/324aeb4c1f50b00a90e12e6579e422242a3146fb) fix: tabs improvements launch review - [`7464a8a`](https://github.com/AppFlowy-IO/AppFlowy/commit/7464a8af6e4a5d2c979c480d53435109cace1412) test: add tests for pin/unpin feature - [`69b2c99`](https://github.com/AppFlowy-IO/AppFlowy/commit/69b2c992d4c8936ba80fdf773333bb2686c446d1) fix: failing test + cleanup ### 📊 Changes **15 files changed** (+488 additions, -168 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/uncategorized/tabs_test.dart` (+136 -39) 📝 `frontend/appflowy_flutter/lib/plugins/ai_chat/chat.dart` (+6 -1) 📝 `frontend/appflowy_flutter/lib/plugins/blank/blank.dart` (+4 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/tab_bar/tab_bar_view.dart` (+5 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database_document/database_document_plugin.dart` (+6 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/document.dart` (+6 -1) 📝 `frontend/appflowy_flutter/lib/plugins/trash/trash.dart` (+5 -4) 📝 `frontend/appflowy_flutter/lib/workspace/application/tabs/tabs_bloc.dart` (+120 -16) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/home_stack.dart` (+11 -4) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/workspace/_sidebar_workspace_menu.dart` (+7 -8) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/navigation.dart` (+5 -5) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/tabs/flowy_tab.dart` (+140 -81) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/tabs/tabs_manager.dart` (+3 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/widgets/tab_bar_item.dart` (+30 -5) 📝 `frontend/resources/translations/en.json` (+4 -1) </details> ### 📄 Description - [x] Close tab menu popover when it should be closed - [x] Make "Close"-option disabled when pinned - [x] Add tooltips to buttons in Tab menu to explain usage or when they're disabled (why) - [x] Close tabs when switching workspace - [x] Move pinned tabs to beginning of tab bar - [x] Include View Icon in tab - [x] Pinned tabs are smaller and only show icon if possible #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [x] I've listed at least one issue that this PR fixes in the description above. - [x] 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:20:29 +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#7626
No description provided.