Skip to content

Infinite scroll #9638

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
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

harshit-dxt
Copy link

Description

I just displayed a loading icon whenever there is loading state.

Refers/Fixes

Fixes: Now shows loading for new grants.

#9616

Testing

I just added an icon to display whenever loading

@harshit-dxt
Copy link
Author

harshit-dxt commented Oct 28, 2021

Added a delay of 2 seconds to show how the icon looked.
The UI looks something like this.
https://user-images.githubusercontent.com/54188077/139319690-a7b97fc8-135b-49ac-8505-cc44c7d7b5a6.mp4

Would love some feedback on this.

@@ -346,7 +346,7 @@
</div>
</div>

<div class="col-12 mb-3 mx-auto col-md-6 text-center" v-else>
<div class="col-12 mb-3 mx-auto col-md-6 text-center" v-else-if="!lock && grants && grants.length == 0">
Copy link
Contributor

Choose a reason for hiding this comment

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

what does the !lock do here ?

Copy link
Author

Choose a reason for hiding this comment

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

!lock ensures that as long as the fetchGrants() is working it should not display the obiwan-bot saying that there exists no other grant.

@thelostone-mc
Copy link
Contributor

@PixelantDesign + @frankchen07 could you have a look at this

@harshit-dxt
Copy link
Author

harshit-dxt commented Nov 2, 2021

Gentle Reminder @thelostone-mc @PixelantDesign.

@harshit-dxt
Copy link
Author

@@ -359,6 +359,10 @@ <h1 class="font-title-lg font-weight-semibold mt-4 mb-2">{% trans "This is not t
</button>
</div>
</div>

<div class="col-12 mb-3 mx-auto col-md-6 text-center text-center" v-show="lock">
<img class="w-100" src="https://s.gitcoin.co/static/v2/images/loading_v2.f27849139c28.gif">
Copy link
Contributor

Choose a reason for hiding this comment

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

Absolute URLs should be hardcoded. Please us "{% static ... %}"

Copy link
Author

Choose a reason for hiding this comment

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

Changed this to "{% static %}" format.

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.

3 participants