[PR #2145] [MERGED] feat: Only show the change cover button and delete button when it's hovering. #5124

New issue

Closed

opened 2026-03-23 22:17:20 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 22:17:20 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/2145
Author: @gitstart
Created: 3/30/2023
Status: ✅ Merged
Merged: 5/3/2023
Merged by: @LucasXu0

Base: main ← Head: FLOWY-OSS-3

📝 Commits (5)

63b4d75 feat: Only show the change cover button and delete button when it's hovering.
bf53116 Merge commit '92f980347f' into FLOWY-OSS-3
f28c786 feat: Only show the change cover button and delete button when it's hovering.
6265980 Merge commit '782d01274d' into FLOWY-OSS-3
9bcfd5a feat: Only show the change cover button and delete button when it's hovering

📊 Changes

4 files changed (+118 additions, -36 deletions)

View changed files

➕ frontend/appflowy_flutter/assets/test/workspaces/cover_image.zip (+0 -0)
➕ frontend/appflowy_flutter/integration_test/cover_image_test.dart (+54 -0)
📝 frontend/appflowy_flutter/integration_test/util/data.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/plugins/cover/cover_node_widget.dart (+62 -35)

📄 Description

fixes #2055

The change cover button and the delete button always show above the cover widget now they will only show when the mouse is hovering on top of the cover image.

Issue Video

https://www.loom.com/share/1e9be9d35de946d5a3d787b62bc9edd8

Fix Video

https://www.loom.com/share/4a59841c6eff48eaaeec1ed60f38bc7e

This code was written and reviewed by GitStart Community. Growing future engineers, one PR at a time.

_{🔄 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/2145 **Author:** [@gitstart](https://github.com/gitstart) **Created:** 3/30/2023 **Status:** ✅ Merged **Merged:** 5/3/2023 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `FLOWY-OSS-3` --- ### 📝 Commits (5) - [`63b4d75`](https://github.com/AppFlowy-IO/AppFlowy/commit/63b4d758b84e6c2c6162d75ccf37cbf3a853fbc7) feat: Only show the change cover button and delete button when it's hovering. - [`bf53116`](https://github.com/AppFlowy-IO/AppFlowy/commit/bf531168d6ca943c99c7c6e4b940937d0fd9f911) Merge commit '92f980347f2acf1b11a37ee5b84fd26d05835eec' into FLOWY-OSS-3 - [`f28c786`](https://github.com/AppFlowy-IO/AppFlowy/commit/f28c78668dfd8130707f08231bcfb0279a0cc8d4) feat: Only show the change cover button and delete button when it's hovering. - [`6265980`](https://github.com/AppFlowy-IO/AppFlowy/commit/6265980222ad36166ef94d6efddcbbfdc0b48a49) Merge commit '782d01274db656798fd255380bf2be88c1f8eec8' into FLOWY-OSS-3 - [`9bcfd5a`](https://github.com/AppFlowy-IO/AppFlowy/commit/9bcfd5aecbac85c769121a8651ea1c1e53e850d6) feat: Only show the change cover button and delete button when it's hovering ### 📊 Changes **4 files changed** (+118 additions, -36 deletions) <details> <summary>View changed files</summary> ➕ `frontend/appflowy_flutter/assets/test/workspaces/cover_image.zip` (+0 -0) ➕ `frontend/appflowy_flutter/integration_test/cover_image_test.dart` (+54 -0) 📝 `frontend/appflowy_flutter/integration_test/util/data.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/plugins/cover/cover_node_widget.dart` (+62 -35) </details> ### 📄 Description fixes #2055 The change cover button and the delete button always show above the cover widget now they will only show when the mouse is hovering on top of the cover image. ### Issue Video https://www.loom.com/share/1e9be9d35de946d5a3d787b62bc9edd8 ### Fix Video https://www.loom.com/share/4a59841c6eff48eaaeec1ed60f38bc7e --- This code was written and reviewed by GitStart Community. Growing future engineers, one PR at a time. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>