Skip to content

fix(files): Add rounded corners back in grid view - #46373

Merged
susnux merged 1 commit into
masterfrom
artonge/fix/rounded_corner_in_grid_view
Jul 11, 2024
Merged

fix(files): Add rounded corners back in grid view#46373
susnux merged 1 commit into
masterfrom
artonge/fix/rounded_corner_in_grid_view

Conversation

@artonge

@artongeartonge commented Jul 9, 2024

Copy link
Copy Markdown
Collaborator

Was removed intentionally due to mockups.

BeforeAfter
imageimage

@artonge
artonge requested a review from skjnldsv as a code ownerJuly 9, 2024 12:00
@artongeartonge self-assigned this Jul 9, 2024
@artongeartonge added this to the Nextcloud 30 milestone Jul 9, 2024
@artonge
artongeforce-pushed the artonge/fix/rounded_corner_in_grid_view branch from 9ac6d38 to 54f48d3CompareJuly 9, 2024 12:02
@artonge
artonge requested a review from susnuxJuly 9, 2024 12:04

@szaimenszaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jancborchardtjancborchardt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

var(--border-radius) for the image preview itself looks good, but the outside element (the hover/focus background) should have a bigger radius to match, so best use var(--border-radius-large) for that.

This is the same how it is in the action menus for example.

@artonge
artongeforce-pushed the artonge/fix/rounded_corner_in_grid_view branch from 54f48d3 to 9ebac30CompareJuly 11, 2024 08:13
@artonge

Copy link
Copy Markdown
CollaboratorAuthor

Updated

@jancborchardtjancborchardt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✨ 🟦

Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge
artongeforce-pushed the artonge/fix/rounded_corner_in_grid_view branch from 9ebac30 to 5fc79e4CompareJuly 11, 2024 12:35
@susnux
susnux merged commit 19c8c63 into masterJul 11, 2024
@susnux
susnux deleted the artonge/fix/rounded_corner_in_grid_view branch July 11, 2024 15:56
@blizzzblizzz mentioned this pull request Jul 24, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants

@artonge@jancborchardt@susnux@szaimen