Task: Implement effective infinite scroll on the new home page

[errorassassin]

Bugheist is adapting a new homepage with all the bugs listed on the homepage. Currently, all the bugs load without even scrolling.
The new home page is currently hosted at https://www.bugheist.com/newhome.

[sidling1]

Hey @errorassassin ,

The issue is that in views.py , we gather all the bugs in one go from the database, but we want it to happen as the user scrolls forward in the home page ?

Is this what is to be implemented ?

[errorassassin]

Hey @errorassassin ,

The issue is that in views.py , we gather all the bugs in one go from the database, but we want it to happen as the user scrolls forward in the home page ?

Is this what is to be implemented ?

Yes, that would improve the user experience a lot.

[sidling1]

Can i work on this issue ?

[CodeWithBishal]

Sure, Feel free to assign the issue to yourself using /assign.

P.S. also make sure the url changes after the pagination, e.g. after scroll when the page 2 is being loaded URL should contain "?page=2" hence it is easier to share a link to someone else with a specific page number.

@sidling1

[nick2432]

/assign

[aakash2330]

please correct me if I'm wrong @CodeWithBishal , we are here trying to remove the pagination bar and implement it by scrolling through the page instead ?

[CodeWithBishal]

@aakash2330 yes correct

[aakash2330]

/assign

[Prakhar-Shankar]

/assign

[Prakhar-Shankar]

@errorassassin How can we navigate to newhome page locally ?

[errorassassin]

just go to [base url]/newhome which would be localhost:8000/newhome by default

[Prakhar-Shankar]

Got it thanks.

[HanilJain]

/assign

[HanilJain]

bugs in newhome which have are of different user and is_hidden=true they must be listed in the page or not.
https://github.com/OWASP/BLT/blob/80702b0c02e68aec123bc90685846c74118a75dd/website/views.py#L198C1-L199C1
here the bugs uses & operator which leads to list all but I think there must be an | operator there. am I correct ?

[errorassassin]

bugs in newhome which have are of different user and is_hidden=true they must be listed in the page or not. https://github.com/OWASP/BLT/blob/80702b0c02e68aec123bc90685846c74118a75dd/website/views.py#L198C1-L199C1 here the bugs uses & operator which leads to list all but I think there must be an | operator there. am I correct ?

No! Replacing the and with or would hide every Issue that's not mine as the second condition would be true whenever the Issue is not mine (mine means the user seeing the homepage). We want to hide all the Issues that are hidden and not mine.
Also, your way of creating an infinite scroll homepage is not effective as it loads all bugs on the first load of the website, even when the user is not scrolling down. We want to load the bugs as the user scrolls down, one page at a time.

[HanilJain]

thanks, got it.
making the required changes.

[HanilJain]

I fixed the issue in this commit db9d344

[CodeWithBishal]

Hi @HanilJain,

If you have created a PR for this commit feel free to mention it with #[PR-ID]

[HanilJain]

fixed #1672

[HanilJain]

@errorassassin Hi can you close this issue
#1672
PR is merged !

[errorassassin]

@HanilJain I dont think that was quite as expected, the page still loads all the bugs on the initial reload. Once again, what I had in my mind was something similar to twitter, loading content as the user scrolls down. Loading all the bugs initially has massively increased the loading time too.

[errorassassin]

observe the scrollbar on these social media websites like twitter

[HanilJain]

@errorassassin Hi thanks for informing.
it was small change in the code
I have resolved in the #1767

[HanilJain]

@errorassassin I think it seems correct to me now. as new content can be seen loading and also the scroll bar.
are they any more change required?

[DonnieBLT]

Let's revert this please. I like the pagination better.