Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#40304 closed defect (bug) (fixed)

PressThis: Incorrect icon for "Chat"

Reported by: ejner69's profile ejner69 Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.8 Priority: normal
Severity: normal Version:
Component: Press This Keywords: has-screenshots has-patch
Focuses: ui Cc:

Description

The icon for "Chat" is the same of the "Standard" in the section Post Format (Press This)

Attachments (2)

Captura de pantalla (6).png (246.6 KB) - added by ejner69 7 years ago.
40304.patch (386 bytes) - added by menakas 7 years ago.
Added dashboards icons for Chat and Statuses in the CSS file

Download all attachments as: .zip

Change History (7)

#1 @afercia
7 years ago

  • Component changed from Post Formats to Press This
  • Focuses administration removed
  • Keywords has-screenshots needs-patch added
  • Milestone changed from Awaiting Review to 4.8

As far as I see, Press This explicitly sets icons for just 8 post formats (including the standard one) and then fallbacks to the standard icon. However, there are 10 default post formats:

wp-admin/css/press-this.css

standard
aside
audio
gallery
image
link
quote
video

wp-includes/post-formats.php

standard
aside
audio
chat    <-- missing
gallery
image
link
quote
status  <-- missing
video

@menakas
7 years ago

Added dashboards icons for Chat and Statuses in the CSS file

#2 @menakas
7 years ago

  • Keywords has-patch added; needs-patch removed

#3 @SergeyBiryukov
7 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 40355:

Press This: Add missing icons for Chat and Status post formats.

Props menakas, ejner69.
Fixes #40304.

#4 @SergeyBiryukov
7 years ago

In 40356:

Press This: Reorder post format icon styles for consistency with get_post_format_strings().

See #40304.

This ticket was mentioned in Slack in #core by menakas. View the logs.


7 years ago

Note: See TracTickets for help on using tickets.