On 10/07/18 at 09:01pm, coderkun wrote:

Add the “rel” attribute for “prev” and “next” to the pagination links of news and packages.

Interesting patch, but does any browser even care about these rel tags these days? A quick search didn't seem to show it's very useful.

--- templates/news/paginator.html | 4 ++-- templates/packages/search_paginator.html | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/templates/news/paginator.html b/templates/news/paginator.html index 57fbeb1..524c666 100644 --- a/templates/news/paginator.html +++ b/templates/news/paginator.html @@ -4,7 +4,7 @@ <p class="news-nav"> {% if page_obj.has_previous %} < Prev</a> + title="Go to previous page" rel="prev">< Prev</a> {% endif %} {% for num in paginator.page_range %} {% ifequal num page_obj.number %} @@ -15,7 +15,7 @@ {% endfor %} {% if page_obj.has_next %} Next ></a> + title="Go to next page" rel="next">Next ></a> {% endif %} </p> </div> diff --git a/templates/packages/search_paginator.html b/templates/packages/search_paginator.html index a748d26..a676626 100644 --- a/templates/packages/search_paginator.html +++ b/templates/packages/search_paginator.html @@ -7,7 +7,7 @@ <span class="prev"> {% if page_obj.has_previous %} < Prev</a> + title="Go to previous page" rel="prev">< Prev</a> {% else %} < Prev {% endif %} @@ -15,7 +15,7 @@ <span class="next"> {% if page_obj.has_next %} Next ></a> + title="Go to next page" rel="next">Next ></a> {% else %} Next > {% endif %} -- 2.19.1

-- Jelle van der Waa