Make WordPress Core

Opened 19 months ago

Closed 4 months ago

Last modified 4 months ago

#54509 closed defect (bug) (fixed)

Attachment edit view details information formatting/styling inconsistency

Reported by: szaqal21's profile szaqal21 Owned by: joedolson's profile joedolson
Milestone: 6.2 Priority: normal
Severity: normal Version:
Component: Media Keywords: has-screenshots has-patch commit
Focuses: ui Cc:

Description (last modified by sabernhardt)

There is some inconsistency in attachment edit view (list and grid style view). In classic edit view (post.php?post=XXX&action=edit) "Original image" link isn't wrapped in <strong> but "Uploaded by" link is. And in edit view of grid style view (dialog) "Original image" label isn't wrapped in <strong> like other labels are.

Attachments (8)

attachment-edit-view.jpg (183.2 KB) - added by szaqal21 19 months ago.
Original image link not wrapped in <strong>
attachment-edit-view-2.jpg (118.8 KB) - added by szaqal21 19 months ago.
Dialog view, Original image label not wrapped in <strong> like other labels.
insert-media-into-post.png (138.4 KB) - added by sabernhardt 6 months ago.
dialog for inserting an image into a post (which does not seem to need a strong tag)
54509.diff (1.2 KB) - added by sabernhardt 6 months ago.
adding strong tags to Attachment details dialog and Edit Media page
Attachment-details-dialog-with-patch.png (149.8 KB) - added by sabernhardt 6 months ago.
Edit-Media-with-patch.png (12.1 KB) - added by sabernhardt 6 months ago.
54509.2.diff (1.2 KB) - added by sabernhardt 6 months ago.
moves strong tag inside link on Edit Media page
Edit-Media-with-patch-54509.2.png (12.2 KB) - added by sabernhardt 6 months ago.

Download all attachments as: .zip

Change History (20)

@szaqal21
19 months ago

Original image link not wrapped in <strong>

@szaqal21
19 months ago

Dialog view, Original image label not wrapped in <strong> like other labels.

#1 @audrasjb
19 months ago

  • Keywords needs-screenshots needs-patch added
  • Version 5.8.2 deleted

#2 @audrasjb
19 months ago

  • Keywords has-screenshots added; needs-screenshots removed

@sabernhardt
6 months ago

dialog for inserting an image into a post (which does not seem to need a strong tag)

#3 @sabernhardt
6 months ago

  • Description modified (diff)
  • Milestone changed from Awaiting Review to 6.2

Thanks for the report! These should be (more) consistent.

Last edited 6 months ago by sabernhardt (previous) (diff)

@sabernhardt
6 months ago

adding strong tags to Attachment details dialog and Edit Media page

#4 @sabernhardt
6 months ago

  • Keywords has-patch added; needs-patch removed

@sabernhardt
6 months ago

moves strong tag inside link on Edit Media page

#5 @sabernhardt
6 months ago

The strong tag is inside the "Uploaded by" link, so the patch is updated to do the same for the new tag inside the "Original image" link.

This ticket was mentioned in Slack in #core-media by antpb. View the logs.


4 months ago

#7 @joedolson
4 months ago

  • Owner set to joedolson
  • Status changed from new to reviewing

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


4 months ago
#8

Make links and labels consistent on media attachment screens

Trac ticket: https://core.trac.wordpress.org/ticket/54509

#9 @joedolson
4 months ago

  • Keywords commit added

#10 @joedolson
4 months ago

  • Status changed from reviewing to accepted

#11 @joedolson
4 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 55187:

Media: Fix inconsistent font weight in media detail views.

Make font weight consistent between different screens and different parts of media views.

Props szaqal21, sabernhardt.
Fixes #54509.

@joedolson commented on PR #3960:


4 months ago
#12

Fixed in r55187

Note: See TracTickets for help on using tickets.