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

Reduced view for completed grants at Homepage + view all button #51

Merged
merged 6 commits into from
Mar 1, 2024

Conversation

Pabl0cks
Copy link
Member

Description

Added a new page to show all completed grants, and a reduced view (via prop) to show at Homepage.

Reduced view has a max limit of 8 grants, and then manually forcing on every breakdown to load 2 rows of cards, except on mobile where is loading 3 (2 cards felt too little). Couldn't think of an cleaner way to implement, and 8 cards felt huge on mobile-tablet.

In the future we can make a similar approach for Ecosystem impact grants, or go for a carousel there instead if the number feels optimal for it.

Closes #36

@Pabl0cks Pabl0cks changed the title Reduced completed grants Reduced view for completed grants Feb 29, 2024
@Pabl0cks Pabl0cks changed the title Reduced view for completed grants Reduced view for completed grants at Homepage + view all button Feb 29, 2024
Copy link

vercel bot commented Feb 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
grants-bg ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 29, 2024 10:02pm

Copy link
Member

@technophile-04 technophile-04 left a comment

Choose a reason for hiding this comment

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

Tysm @Pabl0cks this is great !!!

max limit of 8 grants, and then manually forcing on every breakdown to load 2 rows of cards, except on mobile where is loading 3 (2 cards felt too little). Couldn't think of an cleaner way to implement, and 8 cards felt huge on mobile-tablet.

I am not sure either what's the best solution would be to handle this, but your solution looks and works nicely !!


Just thought came to mind, In very late future we could handle this in backend too using firebase limit so that we only query 8 documents on home page....but lol not at all required to do now !

@Pabl0cks
Copy link
Member Author

Thanks for the review @technophile-04!!

Just thought came to mind, In very late future we could handle this in backend too using firebase limit so that we only query 8 documents on home page....but lol not all required now !

Ohh didn't know that limit existed in firebase, makes sense! ShivGPT > chatGPT 😄🙌

@Pabl0cks Pabl0cks merged commit bb6dc6f into main Mar 1, 2024
3 checks passed
@Pabl0cks
Copy link
Member Author

Pabl0cks commented Mar 1, 2024

Merging this one to create a new PR to iterate on the completed grants design, any changes regarding this PR can be tackled there 🙌

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.

Completed grants needs a carousel or multiple rows
2 participants