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

Fixed Preloader #425

Merged
merged 3 commits into from
Aug 10, 2023
Merged

Fixed Preloader #425

merged 3 commits into from
Aug 10, 2023

Conversation

Nishitbaria
Copy link
Contributor

GitHub Pull Request

Fixes ISSUE #418
Description:

I have fixed the issue with the preloader in the DevEmpire website by correcting the image path that was causing the preloader to malfunction. The preloader was failing to display due to an incorrect image path reference. This pull request addresses the issue and ensures that the preloader is displayed as expected.

Changes Made:

I updated the image path in the preloader component to point to the correct location of the loading image. The previous path was causing the preloader to not show up, which has now been resolved by providing the accurate image path.

Testing:

I have tested the preloader thoroughly on different devices and browsers to confirm that the fix has successfully resolved the issue. The preloader now displays correctly during page loading.

Impact:

This fix significantly improves the user experience on the DevEmpire website by ensuring that the preloader works as intended. Users will now see the preloader while the content is being loaded, reducing confusion and enhancing the overall perception of the website's performance.

image

Screenshots:

(Optional: If applicable, you can attach screenshots demonstrating the preloader displaying correctly after the fix.)

I appreciate your review and feedback on this fix. Please let me know if any further adjustments are needed or if there are any concerns related to this pull request.

@netlify
Copy link

netlify bot commented Aug 7, 2023

Deploy Preview for devempire ready!

Name Link
🔨 Latest commit c39a25e
🔍 Latest deploy log https://app.netlify.com/sites/devempire/deploys/64d494e81fb64e00087c383a
😎 Deploy Preview https://deploy-preview-425--devempire.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Nishitbaria
Copy link
Contributor Author

@swapnilsparsh ,please review and merge my pull request i have fixed preloader

@swapnilsparsh
Copy link
Owner

swapnilsparsh commented Aug 8, 2023

@Nishitbaria As told, add percentage also.

@Nishitbaria
Copy link
Contributor Author

@swapnilsparsh , i have added percentage loader ,please review and merge my pull request

@Nishitbaria
Copy link
Contributor Author

@swapnilsparsh , i have added percentage loader ,please review and merge my pull request

@swapnilsparsh
Copy link
Owner

@Nishitbaria Resolve the conflicts.

@Nishitbaria
Copy link
Contributor Author

@swapnilsparsh done

@swapnilsparsh swapnilsparsh merged commit 45f719e into swapnilsparsh:master Aug 10, 2023
4 checks passed
@swapnilsparsh swapnilsparsh added gssoc23 GirlScript Summer of Code 2023 level2 Medium labels Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc23 GirlScript Summer of Code 2023 level2 Medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants