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 %} > <a class="prev" href="?page={{ page_obj.previous_page_number }}" > - title="Go to previous page">< 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 %} > <a class="next" href="?page={{ page_obj.next_page_number }}" > - title="Go to next page">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 %} > <a href="?page={{ page_obj.previous_page_number }}&{{ > current_query }}" > - title="Go to previous page">< 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 %} > <a href="?page={{ page_obj.next_page_number }}&{{ current_query > }}" > - title="Go to next page">Next ></a> > + title="Go to next page" rel="next">Next ></a> > {% else %} > Next > > {% endif %} > -- > 2.19.1 -- Jelle van der Waa
signature.asc
Description: PGP signature