[PR #5317] [MERGED] fix: page style issues #6762

Closed
opened 2026-03-23 23:16:33 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

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

Base: mainHead: fix_page_style_issues


📝 Commits (10+)

  • d409761 fix: 7 emojis per line
  • ca0f1bc fix: remove shadow of the icons using presets cover
  • 6151fa3 fix: expand appbar buttons size
  • 2544a22 fix: save new title name when it changed
  • 6a9648e feat: add cover image preview
  • 051b86b fix: dismiss unsplash and presets panel auto
  • a7da334 feat: add selected color for cover image and layout section
  • df77b8a fix: selected icon size too small
  • 1c12ef8 fix: dismiss page style panel before pushing to emoji and font selector
  • 718f656 chore: update back button icon

📊 Changes

14 files changed (+293 additions, -138 deletions)

View changed files

📝 CHANGELOG.md (+18 -0)
📝 frontend/Makefile.toml (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/application/base/mobile_view_page_bloc.dart (+8 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/base/app_bar/app_bar.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/base/view_page/app_bar_buttons.dart (+10 -6)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/cover/document_immersive_cover.dart (+9 -7)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/unsplash_image_widget.dart (+30 -23)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_cover_image.dart (+146 -60)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_icon.dart (+47 -26)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_layout.dart (+15 -7)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/page_style_bottom_sheet.dart (+1 -1)
📝 frontend/appflowy_flutter/pubspec.yaml (+1 -1)
📝 frontend/resources/flowy_icons/16x/m_app_bar_back.svg (+3 -3)
📝 frontend/resources/translations/en.json (+2 -1)

📄 Description

Feature Preview

  • 7 emojis per line.

  • updated selected icon style.

  • dismiss page style before opening emoji panel.
    Screenshot 2024-05-13 at 10 11 44

  • set app bar button size to 24.0

  • replace the back button icon

  • don't show shadow when using built-in images
    Screenshot 2024-05-13 at 12 19 37

  • add selected color

  • add preview image
    Screenshot 2024-05-13 at 12 20 15


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/5317 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 5/13/2024 **Status:** ✅ Merged **Merged:** 5/13/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix_page_style_issues` --- ### 📝 Commits (10+) - [`d409761`](https://github.com/AppFlowy-IO/AppFlowy/commit/d409761363cc542aaa9a1aeacb504c5aa15b9e99) fix: 7 emojis per line - [`ca0f1bc`](https://github.com/AppFlowy-IO/AppFlowy/commit/ca0f1bc1a820f1399c3f8b4244fb156b3cd6e1bb) fix: remove shadow of the icons using presets cover - [`6151fa3`](https://github.com/AppFlowy-IO/AppFlowy/commit/6151fa33dbcf6371bc2f985562a2935d3ec2af14) fix: expand appbar buttons size - [`2544a22`](https://github.com/AppFlowy-IO/AppFlowy/commit/2544a22aee4c6d56f6b63efc3d3d8a2f8a1b853c) fix: save new title name when it changed - [`6a9648e`](https://github.com/AppFlowy-IO/AppFlowy/commit/6a9648e9d3d2dc3d92579896ab41a2f37ca41fec) feat: add cover image preview - [`051b86b`](https://github.com/AppFlowy-IO/AppFlowy/commit/051b86b2f30bb38d4b1bae3ec00ba18c2f4fd3e0) fix: dismiss unsplash and presets panel auto - [`a7da334`](https://github.com/AppFlowy-IO/AppFlowy/commit/a7da3348d9be01a62018b1842e0402f44545b8a1) feat: add selected color for cover image and layout section - [`df77b8a`](https://github.com/AppFlowy-IO/AppFlowy/commit/df77b8ac37a3eb45724062db74152b25640a7108) fix: selected icon size too small - [`1c12ef8`](https://github.com/AppFlowy-IO/AppFlowy/commit/1c12ef88b58179f3ccafd0c8a95fa8bac7cef86b) fix: dismiss page style panel before pushing to emoji and font selector - [`718f656`](https://github.com/AppFlowy-IO/AppFlowy/commit/718f656369d7c6409cb26cd8db806fbb11e6a469) chore: update back button icon ### 📊 Changes **14 files changed** (+293 additions, -138 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+18 -0) 📝 `frontend/Makefile.toml` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/application/base/mobile_view_page_bloc.dart` (+8 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/base/app_bar/app_bar.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/base/view_page/app_bar_buttons.dart` (+10 -6) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/cover/document_immersive_cover.dart` (+9 -7) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/unsplash_image_widget.dart` (+30 -23) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_cover_image.dart` (+146 -60) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_icon.dart` (+47 -26) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_layout.dart` (+15 -7) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/page_style_bottom_sheet.dart` (+1 -1) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+1 -1) 📝 `frontend/resources/flowy_icons/16x/m_app_bar_back.svg` (+3 -3) 📝 `frontend/resources/translations/en.json` (+2 -1) </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 - [x] 7 emojis per line. - [x] updated selected icon style. - [x] dismiss page style before opening emoji panel. <img width="447" alt="Screenshot 2024-05-13 at 10 11 44" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/2b46cc14-ad54-4691-96c7-c262503c3e26"> - [x] set app bar button size to 24.0 - [x] replace the back button icon - [x] don't show shadow when using built-in images <img width="379" alt="Screenshot 2024-05-13 at 12 19 37" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/13053af8-9f09-42ea-83d9-9344d34a5834"> - [x] add selected color - [x] add preview image <img width="383" alt="Screenshot 2024-05-13 at 12 20 15" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/dca266bf-d1da-4f59-a936-e8410160e007"> <!--- 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:16:33 +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#6762
No description provided.