Skip to content

let user select number of entries per history page #1712

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

Open
jfriedel opened this issue Aug 1, 2017 · 5 comments
Open

let user select number of entries per history page #1712

jfriedel opened this issue Aug 1, 2017 · 5 comments

Comments

@jfriedel
Copy link

jfriedel commented Aug 1, 2017

Some history logs are long (many entries / changesets), and OpenGrok divides them into pages (with certain number of bugs on a single page). If two entries in the history end up on two distinct pages, there is no way how one could compare them (the entries / changesets). Either a drop-down box with entries per page or "show all" button (so all entries are shown on a single page) would be useful. Of course, you're mileage may vary and a better solution can be found..

@vladak
Copy link
Member

vladak commented Aug 1, 2017

drop down box or slider with number of items per page would work for me.

More sophisticated solution would be some search box for history entries.

@vladak vladak changed the title "show all" / "bugs per history tab" let user select number of entries per history page Aug 1, 2017
@vladak
Copy link
Member

vladak commented Aug 1, 2017

This is related to #1049

@tulinkry
Copy link
Contributor

tulinkry commented Aug 2, 2017

If two entries in the history end up on two distinct pages, there is no way how one could compare them

This is not true, it should be working that you can select two entries on different pages.

Either a drop-down box with entries per page...

Current workaround could be adding a n parameter to the url to display greater number of entries.

@jfriedel
Copy link
Author

jfriedel commented Aug 2, 2017

Correct; I was wrong about the selection. Feel free to close this issue as not a bug if the `n' parameter is not worth to implement. Thanks ;)

@vladak
Copy link
Member

vladak commented Aug 2, 2017

I think it's still worthwhile addition - at least for me. I often bring up the history of a file and then search the contents for keywords as opposed to searching the history using the search.

@vladak vladak added the UI label Aug 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants