[PR #3952] [MERGED] chore: use page icon instead of document icon #5968

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/3952
Author: @richardshiue
Created: 11/16/2023
Status: ✅ Merged
Merged: 11/17/2023
Merged by: @LucasXu0

Base: main ← Head: document-icon

📝 Commits (3)

4eb0b42 chore: use original page icon
874cab0 chore: just replace the icon, bye bye documents.svg
635aa12 fix: big facepalm moment with find and replace

📊 Changes

10 files changed (+13 additions, -18 deletions)

View changed files

📝 frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_add_new_page.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_trash_page.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/document.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/database/referenced_database_menu_item.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/header/custom_cover_picker.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/application/view/view_ext.dart (+4 -4)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/import/import_type.dart (+1 -1)
➕ frontend/resources/flowy_icons/16x/document.svg (+3 -0)
➖ frontend/resources/flowy_icons/16x/documents.svg (+0 -4)
➖ frontend/resources/flowy_icons/16x/page.svg (+0 -4)

📄 Description

Feature Preview

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/3952 **Author:** [@richardshiue](https://github.com/richardshiue) **Created:** 11/16/2023 **Status:** ✅ Merged **Merged:** 11/17/2023 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `document-icon` --- ### 📝 Commits (3) - [`4eb0b42`](https://github.com/AppFlowy-IO/AppFlowy/commit/4eb0b42237a5774f9b2fd4daa04892f7bd2c8cf2) chore: use original page icon - [`874cab0`](https://github.com/AppFlowy-IO/AppFlowy/commit/874cab03c28eaf455b47c6ac5419aa49c9b40521) chore: just replace the icon, bye bye documents.svg - [`635aa12`](https://github.com/AppFlowy-IO/AppFlowy/commit/635aa12c73cc07877b2759380a268c958f13981f) fix: big facepalm moment with find and replace ### 📊 Changes **10 files changed** (+13 additions, -18 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_add_new_page.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_trash_page.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/document.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/database/referenced_database_menu_item.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/header/custom_cover_picker.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/application/view/view_ext.dart` (+4 -4) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/import/import_type.dart` (+1 -1) ➕ `frontend/resources/flowy_icons/16x/document.svg` (+3 -0) ➖ `frontend/resources/flowy_icons/16x/documents.svg` (+0 -4) ➖ `frontend/resources/flowy_icons/16x/page.svg` (+0 -4) </details> ### 📄 Description <img width="248" alt="image" src="https://github.com/AppFlowy-IO/AppFlowy/assets/71320345/78a52b3f-d48d-4457-80d2-cee721440ff1"> ### Feature Preview  ---  #### PR Checklist - [ ] 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>