[GH-ISSUE #5318] [FR] Add a border to the selected Unsplash image. #2357

Closed
opened 2026-03-23 21:21:43 +00:00 by mirror · 3 comments
Owner

Originally created by @LucasXu0 on GitHub (May 13, 2024).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/5318

Description

Currently, only the built-in images are given a border when selected. We should also add a border to selected Unsplash images.

Screenshot 2024-05-13 at 13 34 50 Screenshot 2024-05-13 at 13 33 39

Impact

Make the UI consistent.

Additional Context

Related file: https://github.com/AppFlowy-IO/AppFlowy/tree/main/frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_cover_image.dart

Originally created by @LucasXu0 on GitHub (May 13, 2024). Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/5318 ### Description Currently, only the built-in images are given a border when selected. We should also add a border to selected Unsplash images. <img width="402" alt="Screenshot 2024-05-13 at 13 34 50" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/80752ecb-9208-4019-818d-24364d010177"> <img width="553" alt="Screenshot 2024-05-13 at 13 33 39" src="https://github.com/AppFlowy-IO/AppFlowy/assets/11863087/7ffd26f0-7d77-401c-947d-cd2fff578b6a"> ### Impact Make the UI consistent. ### Additional Context Related file: https://github.com/AppFlowy-IO/AppFlowy/tree/main/frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/page_style/_page_style_cover_image.dart
Author
Owner

@sudhar08 commented on GitHub (May 19, 2024):

Hey @LucasXu0 I can't understand what you are trying to do. so can you explain a bit more clearly it was very important to build a new feature for this community.

<!-- gh-comment-id:2119291115 --> @sudhar08 commented on GitHub (May 19, 2024): Hey @LucasXu0 I can't understand what you are trying to do. so can you explain a bit more clearly it was very important to build a new feature for this community.
Author
Owner

@Xazin commented on GitHub (May 19, 2024):

Hey @LucasXu0 I can't understand what you are trying to do. so can you explain a bit more clearly it was very important to build a new feature for this community.

In the first image that Lucas attached to the issue report, under the Background image category, you see that the selected image has a blue border with some spacing and then the image.

Now in the second image attached, you see how selecting an Unsplash image looks, but if the unsplash image is already the selected image, there is no blue border indicating this.

This issue aims to address this discrepancy, by adding the same border from the first picture, to the unsplash image if it is the currently used (selected) one.

<!-- gh-comment-id:2119349323 --> @Xazin commented on GitHub (May 19, 2024): > Hey @LucasXu0 I can't understand what you are trying to do. so can you explain a bit more clearly it was very important to build a new feature for this community. In the first image that Lucas attached to the issue report, under the `Background image` category, you see that the selected image has a blue border with some spacing and then the image. Now in the second image attached, you see how selecting an Unsplash image looks, but if the unsplash image is already the selected image, there is no blue border indicating this. This issue aims to address this discrepancy, by adding the same border from the first picture, to the unsplash image if it is the currently used (selected) one.
Author
Owner

@sudhar08 commented on GitHub (May 20, 2024):

@Xazin Thank you for explaining

<!-- gh-comment-id:2119587094 --> @sudhar08 commented on GitHub (May 20, 2024): @Xazin Thank you for explaining
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#2357
No description provided.