Make WordPress Core

Opened 18 months ago

Last modified 18 months ago

#58391 new feature request

Ugly looking file editing section

Reported by: ahsannayem's profile ahsannayem Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Media Keywords: has-patch needs-design-feedback has-screenshots
Focuses: ui, css Cc:

Description

While editing (adding title, description)a file, the view looks ugly.

Here is the screenshot: https://prnt.sc/B3XPrV9f0Rsx

If we add some border radius, it would look nice.

Thank you

Attachments (4)

bugggg.png (438.2 KB) - added by ahsannayem 18 months ago.
after patch.png (457.4 KB) - added by ahsannayem 18 months ago.
After Patch Result
2023-05-24 13_07_15-Mediathek ‹ DEV — WordPress.png (13.1 KB) - added by Presskopp 18 months ago.
hr.png (8.6 KB) - added by Presskopp 18 months ago.

Download all attachments as: .zip

Change History (13)

@ahsannayem
18 months ago

This ticket was mentioned in PR #4497 on WordPress/wordpress-develop by @ahsannayem.


18 months ago
#1

  • Keywords has-patch added

#2 @audrasjb
18 months ago

  • Component changed from General to Media
  • Focuses ui css added
  • Keywords needs-design-feedback added
  • Version 6.2.2 deleted

Hello, welcome to Trac and thanks for the ticket and patch,

First, "ugly" sounds very opinionated.
I'm not sure adding a 10px border radius would really improve this modal.

Adding needs-design-feedback so the Design team can take a look on this patch, but honestly I'm not sure about the relevancy of this proposal 🙂

#3 @mukesh27
18 months ago

Hi there! thanks for ticket and PR.

Same thought as @audrasjb: border radius did improve the design.

#4 @ahsannayem
18 months ago

Hello @audrasjb thank you for your comment.

I have been using WordPress for a long time but while working on this section, it would be nice to have a nice radius on there. I have shared my opinion with my college about this and he also agreed. Which is why I raised this ticket.

Waiting for the Design Feeedback 😊

Thank you

#5 @SergeyBiryukov
18 months ago

Hi there, welcome to WordPress Trac!

Could you share a screenshot after the patch as well?

@ahsannayem
18 months ago

After Patch Result

#6 @alamgircsebd
18 months ago

Here can we make the above border full width, I think it will be more looks good.

Here is the reference https://prnt.sc/EH-5EgLmA7B9

#7 @Presskopp
18 months ago

The border is not full width because the box isn't. Why? To leave some space for the spinner.

#8 @Presskopp
18 months ago

If we would remove the border-bottom from the details class and just add a <hr> after it, and then give the following settings class a margin-top: 15px; it would look like that:

@Presskopp
18 months ago

#9 @Presskopp
18 months ago

  • Keywords has-screenshots added
Note: See TracTickets for help on using tickets.