Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#32477 closed defect (bug) (invalid)

index.php@Twenty_Fifteen : There is an unnecessary `space`.

Reported by: tmatsuur Owned by:
Priority: normal Milestone:
Component: Bundled Theme Version: 4.2
Severity: normal Keywords:
Cc: Focuses:

Description

It will become a fine thing, but I will report.
The location is 48 line of index.php.
Space before end tag of span I think unnecessary.

Now:

'before_page_number' => '<span class="meta-nav screen-reader-text">' . __( 'Page', 'twentyfifteen' ) . ' </span>',

I think:

'before_page_number' => '<span class="meta-nav screen-reader-text">' . __( 'Page', 'twentyfifteen' ) . '</span>',

Change History (2)

#1 @ocean90
11 years ago

  • Component ThemesBundled Theme
  • Milestone Awaiting Review
  • Resolutioninvalid
  • Status newclosed
  • Version 4.2.24.2

Hello tmatsuur, thanks for your report.

Please take a look at paginate_links() where you'll see this line:
$page_links[] = "<span class='page-numbers current'>" . $args['before_page_number'] . number_format_i18n( $n ) . $args['after_page_number'] . "</span>";

The space is required to separate the screen reader text and the page number.

#2 @tmatsuur
11 years ago

Thank you for reply.
I was able to convince.

Note: See TracTickets for help on using tickets.