Opened 2 years ago

Closed 2 years ago

#17385 closed defect (bug) (fixed)

Post time is missing in Twenty Eleven

Reported by: peaceablewhale Owned by: nacin
Priority: normal Milestone: 3.2
Component: Bundled Theme Version: 3.2
Severity: normal Keywords: has-patch
Cc: lancewillett, iandstewart, mike.schroder@…

Description

Twenty Eleven only shows post date, but the time is missing.

Attachments (8)

17385.diff (12.9 KB) - added by sorich87 2 years ago.
17385.escape-output.diff (12.8 KB) - added by mfields 2 years ago.
escacpe output. as mentioned in #17563
author-link-hiding.diff (2.1 KB) - added by iandstewart 2 years ago.
17385.2.diff (20.7 KB) - added by DH-Shredder 2 years ago.
Combination of Newest patches here and #17563, with additional esc_url as appropriate
17385.3.diff (20.7 KB) - added by DH-Shredder 2 years ago.
Added rel="author" to twentyeleven_posted_on
17385.4.diff (21.1 KB) - added by iandstewart 2 years ago.
Add .by-author class to the footer of content-image.php
17385.5.diff (21.0 KB) - added by iandstewart 2 years ago.
Use the less cryptic display:none to hide author bits
17385.6.diff (1.1 KB) - added by aaroncampbell 2 years ago.

Download all attachments as: .zip

Change History (25)

  • Cc lancewillett iandstewart added
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 3.2

It was displayed via the title attribute in Twenty Ten.

To avoid code duplication we should also create twentyeleven_posted_on() (see twentyten_posted_on()).

  • Keywords has-patch added; needs-patch removed
  • Owner set to nacin
  • Status changed from new to reviewing

Thanks for the patch!

The patch looks good to me.

Last edited 2 years ago by iandstewart (previous) (diff)

mfields2 years ago

escacpe output. as mentioned in #17563

author-link-hiding.diff builds on the above patches and adds support for only displaying author links if ismulti_author() returns true.

  1. Would display: none make more sense and be less cryptic?
  1. Why not just hide author in PHP using is_multi_author()?

Display none would be less cryptic and PHP simpler but I'd prefer to keep it there for machines and bots to read. I'm open to either though if it means hiding the name for single-author blogs.

  • Component changed from Themes to Bundled Theme

We shouldn't hide in PHP but using CSS so a child theme can easily override the decision.

  • Keywords needs-refresh added

Good call on the CSS. Sounds like we just need a patch refresh here. display:none + implements the new function everywhere.

Combination of Newest patches here and #17563, with additional esc_url as appropriate

Seems that twentyeleven_posted_on() is missing rel="author"

Added rel="author" to twentyeleven_posted_on

  • Cc mike.schroder@… added
  • Keywords needs-refresh removed

The CSS author hiding if ! is_multi_author() is working as expected for me. As is, resetting with CSS in a child theme.

Last edited 2 years ago by iandstewart (previous) (diff)

Add .by-author class to the footer of content-image.php

Use the less cryptic display:none to hide author bits

In [18244]:

Twenty Eleven: escaping frenzy, hide authors if a single author blog. props DH-Shredder, props iandstewart, see #17385, fixes #17563.

In [18257]:

Remove extra </a>. props aaroncampbell. see #17385.

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

Any more escaping can be done on the audit ticket, #17748.

Note: See TracTickets for help on using tickets.