Context Navigation

← Previous Ticket
Next Ticket →

#24519 closed task (blessed) (fixed)

Add Post Format icons to the old radio chooser

Reported by:	markjaquith	Owned by:	helen
Priority:	high	Milestone:	3.6
Component:	Post Formats	Version:	3.6
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

Little post format icons should be added to the old radio chooser, like MelChoyce mocked up:

Attachments (2)

24519.patch (4.0 KB ) - added by ryelle 13 years ago.
24519.2.patch (4.6 KB ) - added by kovshenin 13 years ago.

Download all attachments as: .zip

Change History (8)

@ryelle
13 years ago

Attachment 24519.patch added

#1 @ryelle
13 years ago

Added spans using the classes from the post list table icons, so I also had to remove the element from the selectors in CSS & add a format-standard class. I also added more space around the options.

#2 @helen
13 years ago

Would it make more sense to change the sprites to be vertical and nix the empty span? I'd also rather add a selector or name it better rather than shoe-horning in one that isn't very semantic to use.

#3 @DrewAPicture
13 years ago

Keywords needs-refresh added

@kovshenin
13 years ago

Attachment 24519.2.patch added

#4 @kovshenin
13 years ago

Keywords has-patch added; needs-refresh removed

Refreshed in 24519.2.patch

#5 @helen
13 years ago

In 24426:

Change the post format icons sprite to be vertical rather than horizontal in order to allow it to be used for wider items such as labels. RTL tweak for the icon in the posts list table. See #16047, #24519.

#6 @helen
13 years ago

Owner set to helen
Resolution → fixed
Status new → closed

In 24429:

Add post format icons to the radio chooser. This also trims empty margins from the sprites to make them usable in RTL and make positioning numbers more sane. props ryelle, kovshenin for the initial patch. fixes #24519.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#24519 closed task (blessed) (fixed)

Add Post Format icons to the old radio chooser

Description

Attachments (2)

Change History (8)

@ryelle 13 years ago

#1 @ryelle 13 years ago

#2 @helen 13 years ago

#3 @DrewAPicture 13 years ago

@kovshenin 13 years ago

#4 @kovshenin 13 years ago

#5 @helen 13 years ago

#6 @helen 13 years ago