Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add widget listing in edit page is not respecting sorting #15486

Closed
fmontes opened this issue Oct 31, 2018 · 3 comments

Comments

@fmontes
Copy link
Member

commented Oct 31, 2018

As a user I want to sort the listing by any of the columns and that pagination respect that sorting.

Expected Behavior

When user sort the table listing and the paginate, the sort order should remain the same

Current Behavior

When user paginate the sort order mess up

Possible Solution

N/A

Known Workarounds

N/A

Steps to Reproduce (for bugs)

  1. Edit a page
  2. Select to add a widget
  3. Sort the table listing by any of the column (title)
  4. Paginate with the button at the bottom
  5. Sort order mess up.

Acceptance Criteria

  1. User should be able to sort and paginate without changing the results sorted
  2. Move language and select button column to the right: Title | Author | Step | Lang | Button
  3. Work in all the supported browsers (don't forget IE11 and iPad)
  4. Multilanguage
  5. Unit test
@alfredo-dotcms

This comment has been minimized.

Copy link
Contributor

commented Nov 26, 2018

looks like a BE issue, when you click on any column header and any button from the pagination, the selected column params are being sent in the request. So there could be an issue in the query that is being done to sort by title.

Note: Sort seems to be working for the other columns

@freddyucv freddyucv self-assigned this Nov 27, 2018

@alfredo-dotcms

This comment has been minimized.

Copy link
Contributor

commented Dec 20, 2018

might be related to #15473 (comment)

fmontes added a commit that referenced this issue Jan 7, 2019

fmontes added a commit that referenced this issue Jan 7, 2019

fmontes added a commit that referenced this issue Jan 7, 2019

@bryanboza bryanboza added this to FOX in QA Jan 10, 2019

@bryanboza

This comment has been minimized.

Copy link
Contributor

commented Jan 10, 2019

Fixed, tested on master // Postgres // FF

@bryanboza bryanboza moved this from FOX to Done in QA Jan 10, 2019

@wezell wezell closed this Jan 31, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.