Skip to content
This repository has been archived by the owner on Nov 19, 2020. It is now read-only.

reduce number of references per page #267

Open
harunurhan opened this issue Dec 1, 2017 · 2 comments
Open

reduce number of references per page #267

harunurhan opened this issue Dec 1, 2017 · 2 comments

Comments

@harunurhan
Copy link
Collaborator

This will improve performance dramatically. Since changing page is easy (thanks to sticky navigation etc.), there is not much point to show more than 20, references IMHO.

We can increase it later, when we fix the performance issue (if it is fixable ofc)

What do you think @StellaCh ?

@StellaCh
Copy link
Contributor

StellaCh commented Dec 1, 2017

20 is too little for the curators who repeatedly ask for as many references in a page as possible.
What is it not performing well?

@harunurhan
Copy link
Collaborator Author

Why load 40 if they can see 3-4 references tops on the screen. I mean if changing the page is that difficult, we can add shortcut or even maybe auto change when scroll at the end of the current page.

Problem is framework limitation eventhough nothing changed on the UI, angular needs to execute something for all components each time when there is an event (like mousing hover an input, clicking on an input, clicking a button etc) so this creates several 100ms lag for each event. IMO this is bad.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants