paginate_links() - Replace CSS class 'page-numbers' with 'page-number'

[antonioeatgoat]

Function paginate_links() uses the CSS class page-numbers on single items, page-number would be more correct. The plural name seems to indicate a wrapping container.

It's indeed the same naming expected by Twenty Seventeen:

wp_link_pages( array(
	'before'      => '<div class="page-links">' . __( 'Pages:', 'twentyseventeen' ),
	'after'       => '</div>',
	'link_before' => '<span class="page-number">',
	'link_after'  => '</span>',
) );

[antonioeatgoat]

Replaced "page-numbers" with "page-number" everywhere, both core and themes

[joyously]

This would affect more themes than just the core themes.
Why change it if it's not broken?

[antonioeatgoat]

It was in naming clarity/consistency sake. However that's totally true, it will upsets other themes and maybe the result doesn't worth the trouble.

Probably it was a consideration to state on slack before open the ticket. If this is the case, I apologize.

[dd32]

I can't see any benefit to renaming this, other than consistency sake.

As the change affects themes, I don't see any need to change this and break a bunch of themes overnight when 5.0 is released. I'm going to close this as wontfix for that reason unfortunately.
We could add a second class here, but that just seems like duplicating it needlessly.