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.

Sign up for GitHub

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

Jump to bottom

Using the pager of the result page does not display the top of the page #2322

Closed
paprickka opened this issue Aug 24, 2021 · 2 comments · Fixed by rero/ng-core#510
Closed

Using the pager of the result page does not display the top of the page #2322

paprickka opened this issue Aug 24, 2021 · 2 comments · Fixed by rero/ng-core#510
Assignees
@Garfield-fr
Labels
bug Breaks something but is not blocking f: public ui Public interface, as opposed to the professional interface p-Medium Medium priority (to be solved within 1-2 years) UX User experience, ergonomy

Comments

@paprickka
Copy link

Describe the bug

In public view, when going through the different pages of a result list should load new page on it"s top.

To Reproduce

  1. Run a search for example on a term like "pendule"
  2. Go to the bottom of the first results page
  3. Click on any page number to go to it
    image
  4. The selected page is displayed, however, we"re not at the top of it

Expected behavior

When changing page, we should be at its top, not at its "somehow" bottom.

Context

  • server: bib.rero.ch
  • version: v1.4.6
  • browser: Firefox 91.0.1
@paprickka paprickka added bug Breaks something but is not blocking triage labels Aug 24, 2021
@iGormilhit iGormilhit added p-Low Low priority f: public ui Public interface, as opposed to the professional interface UX User experience, ergonomy and removed triage labels Aug 25, 2021
@iGormilhit iGormilhit changed the title Results page browsing not being at its top Using the pager of the result page does not display the top of the page Aug 25, 2021
@iGormilhit
Copy link

@paprickka Thanks for this issue!

@ManaDeweerdt
Copy link
Contributor

ManaDeweerdt commented Nov 22, 2021 •
edited by pronguen
Loading

Please add a pager at the top of the result page. Not only at the bottom.

-> PO Group decided to wait with this improvement.

@PascalRepond PascalRepond added p-Medium Medium priority (to be solved within 1-2 years) and removed p-Low Low priority labels Feb 21, 2022
@Garfield-fr Garfield-fr self-assigned this Jul 4, 2022
Garfield-fr added a commit to Garfield-fr/ng-core that referenced this issue Jul 4, 2022
@Garfield-fr
search: move to the top of the page if you change page
f90b96f 
* Closes rero/rero-ils#2322.

Co-Authored-by: Bertrand Zuchuat <[email protected]>
@Garfield-fr Garfield-fr mentioned this issue Jul 4, 2022
Merged
7 tasks
Garfield-fr added a commit to rero/ng-core that referenced this issue Jul 5, 2022
@Garfield-fr
search: move to the top of the page if you change page
c33013e 
* Closes rero/rero-ils#2322.

Co-Authored-by: Bertrand Zuchuat <[email protected]>
Garfield-fr added a commit to rero/ng-core that referenced this issue Jul 6, 2022
@Garfield-fr
search: move to the top of the page if you change page
73a9e61 
* Closes rero/rero-ils#2322.

Co-Authored-by: Bertrand Zuchuat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Garfield-fr Garfield-fr

Labels
bug Breaks something but is not blocking f: public ui Public interface, as opposed to the professional interface p-Medium Medium priority (to be solved within 1-2 years) UX User experience, ergonomy
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

search: move to the top of the page if you change page Garfield-fr/ng-core
5 participants
@Garfield-fr @iGormilhit @ManaDeweerdt @PascalRepond @paprickka