Skip to content

Stick result pagination to top#936

Merged
aeisenberg merged 2 commits intogithub:mainfrom
bmuskalla:stickyPagination
Aug 26, 2021
Merged

Stick result pagination to top#936
aeisenberg merged 2 commits intogithub:mainfrom
bmuskalla:stickyPagination

Conversation

@bmuskalla
Copy link
Copy Markdown

@bmuskalla bmuskalla commented Aug 26, 2021

When navigating pages of results, one had to scroll to top to go to the next page. Making the pagination controls stick to the top of the results editor, they're always at arms reach.

ybR3y4Mo5V

Fixes #526

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • @github/docs-content-codeql has been cc'd in all issues for UI or other user-facing changes made by this pull request.

@bmuskalla bmuskalla requested a review from aeisenberg August 26, 2021 10:55
@bmuskalla bmuskalla requested a review from a team as a code owner August 26, 2021 10:55
@bmuskalla
Copy link
Copy Markdown
Author

Not sure if any of the checklist items is relevant for such a change

Copy link
Copy Markdown
Contributor

@edoardopirovano edoardopirovano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! I would say this probably warrants a CHANGELOG entry as it is visible to users.

@bmuskalla
Copy link
Copy Markdown
Author

Done

@aeisenberg
Copy link
Copy Markdown
Contributor

Nice! Thanks for the PR.

@aeisenberg aeisenberg merged commit f6485da into github:main Aug 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Better UI layout for pagination

3 participants