877239 - In Elastic Search, previous and next buttons are in wrong order

Status: RESOLVED FIXED

(developer.mozilla.org Graveyard :: Search, defect)

Description

[John Karahalis [:openjck]]

What did you do?
================
These steps assume you have the "Elastic Search" flag enabled on your account.

1. Visit https://developer.mozilla.org/en-US/search?q=html
2. Click "Next Page"
3. Scroll to very bottom of page

What happened?
==============
"Next page" button appears to the right of "Previous Page"

What should have happened?
==========================
"Previous page" should appear to the right of "Next page"

Is there anything else we should know?
======================================

Comment 1

[Luke Crouch [:groovecoder]]

I'm confused. The current order is Next Page, Previous Page.

http://screencast.com/t/VGjfWpYk83

So is this fixed now?

Comment 2

[John Karahalis [:openjck]]

David might know more. David, did you change anything here?

If not, I probably just words my messed up in the steps to reproduce. Here is a corrected version.

> What did you do?
> ================
> These steps assume you have the "Elastic Search" flag enabled on your account.
> 
> 1. Visit https://developer.mozilla.org/en-US/search?q=html
> 2. Click "Next Page"
> 3. Scroll to very bottom of page
> 
> What happened?
> ==============
> "Previous page" button appears to the right of "Next Page", like this:
> http://screencast.com/t/VGjfWpYk83
> 
> What should have happened?
> ==========================
> "Next page" should appear to the right of "Previous page", like this or similar:
> http://i.imgur.com/17r2PEj.png

Comment 3

[David Walsh :davidwalsh]

PR in to fix this.

Comment 4

[[github robot]]

Commits pushed to master at https://github.com/mozilla/kuma

https://github.com/mozilla/kuma/commit/ce9b3917b2f87f77c9113a79064dba1750032ad4
fix bug 877239 - Reverse order of 'next' and 'prev' in redesign

https://github.com/mozilla/kuma/commit/443acc946096ff3e8ea4b882f9f0bf82a5f52a97
Merge pull request #1391 from darkwing/redesign-search-pagination-877239

fix bug 877239 - Reverse order of 'next' and 'prev' in redesign