Ticket #615 (closed Patch: wontfix)

Opened 6 months ago

Last modified 6 months ago

patch for digg pagination

Reported by: Edy Owned by: - No owner -
Priority: minor Milestone: 2.2
Component: Libraries Version: SVN HEAD
Keywords: pagination, digg Cc:

Description

I created a patch to match the original digg pagination

Attachments

digg.patch (2.9 kB) - added by Edy 6 months ago.

Change History

Changed 6 months ago by Edy

Changed 6 months ago by Geert

  • status changed from new to closed
  • resolution set to wontfix

Everybody is going to want to do this differently. Let's keep the provided pagination style views as lean as possible. Otherwise we'll end up throwing in spans, divs and classes everywhere.

Moreover, why replace the <strong> element for the current page with a <span class="current">? What about semantic markup?

Changed 6 months ago by Nowaker

I agree with Edy. There should not be <strong>. Why? The discussion for example here: http://www.webmasterworld.com/forum21/9181.htm

The element in pagination is not any important word. It should only be better visible, but doesn't have any important meaning in the text.

Note: See TracTickets for help on using tickets.