💎 PREMIUM: Ticket/ - Complete Album!

Get WordPress

Make WordPress Core

Opened 9 years ago

Closed 8 years ago

#40184 closed enhancement (fixed)

Twenty Seventeen: the tag cloud widget should output a list

Reported by: samikeijonen's profile sami.keijonen Owned by: samikeijonen's profile sami.keijonen
Milestone: 4.9 Priority: normal
Severity: normal Version: 4.7
Component: Bundled Theme Keywords: has-patch tag-cloud needs-testing
Focuses: accessibility Cc:

Description

In ticket #35566 there is new tag cloud widget coming up with accessibility improvements. For bundled themes there is also general ticket #40138 for tag cloud widget.

This is ticket for Twenty Seventeen to output tag cloud widget as a list.

  • It's better for accessibility.
  • Remain the same styles.
  • Remove unnecessary !important rules.

Twenty Sixteen have an example how to use widget_tag_cloud_args for same font size. Let's add $args['format'] = 'list' in there.

Patch is coming up.

Attachments (6)

40184.diff​ (2.2 KB) - added by sami.keijonen 9 years ago.
Tag Cloud Widget.png​ (3.4 KB) - added by lukecavanagh 9 years ago.
Tag Cloud Widget - Twenty Seventeen
40184.1.diff​ (2.2 KB) - added by sami.keijonen 9 years ago.
40184.2.diff​ (2.2 KB) - added by sami.keijonen 8 years ago.
Screen Shot 2017-10-03 at 7.35.45 PM.png​ (26.3 KB) - added by davidakennedy 8 years ago.
Tag Cloud in RTL
40184.2.patch​ (421 bytes) - added by davidakennedy 8 years ago.
Adds RTL styles for new ul based tag cloud.

Download all attachments as: .zip

Change History (35)

@sami.keijonen
9 years ago

This ticket was mentioned in ​Slack in #accessibility by sami.keijonen. ​View the logs.
9 years ago

@lukecavanagh
9 years ago

Tag Cloud Widget - Twenty Seventeen

#2 @lukecavanagh
9 years ago

@sami.keijonen

Patch applies cleanly and works as expected.

#3 @lukecavanagh
9 years ago

  • Keywords has-patch added

#4 @sami.keijonen
9 years ago

Thanks @lukecavanagh for testing!

This ticket was mentioned in ​Slack in #accessibility by sami.keijonen. ​View the logs.
9 years ago

This ticket was mentioned in ​Slack in #accessibility by afercia. ​View the logs.
9 years ago

#7 @afercia
9 years ago

  • Keywords tag-cloud added
  • Milestone changed from Awaiting Review to 4.8

This ticket was mentioned in ​Slack in #accessibility by afercia. ​View the logs.
9 years ago

This ticket was mentioned in ​Slack in #core by jeffpaul. ​View the logs.
9 years ago

#10 @jbpaul17
9 years ago

  • Milestone changed from 4.8 to 4.8.1

Punting to 4.8.1 per today's bug scrub.

This ticket was mentioned in ​Slack in #accessibility by adamsoucie. ​View the logs.
9 years ago

@sami.keijonen
9 years ago

#12 @sami.keijonen
9 years ago

Now that new tag cloud widget is in trunk I added new 40184.1.diff where I only changed version number to 1.3: @since Twenty Seventeen 1.3.

#13 @afercia
9 years ago

  • Milestone changed from 4.8.1 to 4.9

This ticket was mentioned in ​Slack in #accessibility by afercia. ​View the logs.
8 years ago

#15 @afercia
8 years ago

  • Version changed from 4.8 to 4.7

This ticket was mentioned in ​Slack in #accessibility by afercia. ​View the logs.
8 years ago

This ticket was mentioned in ​Slack in #accessibility by sergey. ​View the logs.
8 years ago

This ticket was mentioned in ​Slack in #accessibility by afercia. ​View the logs.
8 years ago

#19 @afercia
8 years ago

  • Owner set to sami.keijonen
  • Status changed from new to assigned

@sami.keijonen
8 years ago

#20 @sami.keijonen
8 years ago

I modified version number in the 40184.2.diff and added space after @param.

Tested that markup changes to list format and design keeps the same.

In the widget settings Show tag counts also works as should.

Screenshot before:

https://cldup.com/ndk1fKnE3V-3000x3000.png

Screenshot after:

https://cldup.com/DI5ReSMKpx.png

#21 @jbpaul17
8 years ago

  • Keywords needs-testing added

This ticket was mentioned in ​Slack in #core by jeffpaul. ​View the logs.
8 years ago

This ticket was mentioned in ​Slack in #accessibility by afercia. ​View the logs.
8 years ago

#24 @SergeyBiryukov
8 years ago

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

In 41703:

Twenty Seventeen: Change tag cloud format to a list (<ul>) for better semantics and accessibility.

List markup allows screen reader users to know in advance how many tags are within the list.

Props sami.keijonen.
Fixes #40184.

This ticket was mentioned in ​Slack in #accessibility by davidakennedy. ​View the logs.
8 years ago

@davidakennedy
8 years ago

Tag Cloud in RTL

#26 @davidakennedy
8 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Thanks for the work here @sami.keijonen and for the merge @SergeyBiryukov!

It looks like this also needs RTL styles since we're floating some things and applying margins. See:

https://core.trac.wordpress.org/attachment/ticket/40184/Screen%20Shot%202017-10-03%20at%207.35.45%20PM.png

I noticed this also in #40138.

@davidakennedy
8 years ago

Adds RTL styles for new ul based tag cloud.

#27 @sami.keijonen
8 years ago

Thanks! I forgot all about the RTL styles.

This ticket was mentioned in ​Slack in #core by jeffpaul. ​View the logs.
8 years ago

#29 @SergeyBiryukov
8 years ago

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

In 41754:

Twenty Seventeen: After [41703], adjust RTL styles for tag cloud widget.

Props davidakennedy.
Fixes #40184.

Note: See TracTickets for help on using tickets.
🔥 TRENDING HENTAI
Updates loading...
© 2024 Sarada Gallery - 18+ Only