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.

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

Update index.html added scroll to top button #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sempijja
Copy link
Contributor

I earlier saw a suggestion to add pagination, though it could improve a couple of things this is a "wall" and should be treated as such without pages. Changes can be made to the button in case before deployment

I earlier saw a suggestion to add pagination, though it could improve a couple of things this is a "wall" and should be treated as such without pages.
Changes can be made to the button in case before deployment
@wkambale
Copy link
Owner

@Musacoli, there's an interesting bit here about pagination. and i think i agree with this perspective. this a wall, not a website.

@Musacoli
Copy link
Collaborator

Musacoli commented Jul 23, 2024

@Musacoli, there's an interesting bit here about pagination. and i think i agree with this perspective. this a wall, not a website.

Fair point with that, though we might also want to consider the bandwidth strain of having no pagination, at least we could use lazy loading versus numbered pagination, that way we maintain a wall and also reduce the weight of the initial loading experience.

Almost like this #66

@kaybrian
Copy link
Collaborator

please could you just resolve the errors there, I like the idea of infinite scrolling with lazy loading

@skapeyi
Copy link
Collaborator

skapeyi commented Jul 25, 2024

@sempijja please update the PR with the latest changes and we'll test and provide feedback

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.

5 participants