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

Changed the Cursor Animation to INDIAN TRICOLOR Theme and ADDED IT TO ALL WEBPAGES (which were without cursor Effect) #1490

Merged
merged 13 commits into from
Oct 31, 2024

Conversation

PatelHarsh2006
Copy link
Contributor

@PatelHarsh2006 PatelHarsh2006 commented Oct 25, 2024

Changed the Cursor Animation to INDIAN TRICOLOR Theme and ADDED IT TO ALL WEBPAGES (Which were Without Cursor Animation Effect)

(BEFORE):
https://github.com/user-attachments/assets/12328a71-d6d7-4714-8c88-fc2e2a493be0

ALSO, So Many Pages were Without Cursor Animation Effect, So also Added Cursor Animation Effect to All Such Pages :
Some of Them Are :

20241025_134524
20241025_134540
20241025_134557
20241025_134842

 AFTER:
YouCut_20241025_142347747.mp4

High Quality Video:
https://github.com/user-attachments/assets/a52c57ae-fc1a-4fe5-8a6e-97e34a18f080


Mam, Please Don't Close the Pull Request if it Shows RESOLVE CONFLICTS, Instead Reply ME in Such Case and Give Me Few Hours of Time in case I haven't Seen it, for Resolving Conflicts, and I will Reply Back when resolved conflicts and then Merge it, but Please don't close it. ( Bcz we just have to remove <<<<< and ------ kind of tags that github by errors takes in the commited file, so i will remove it, if it happens !)




Related Issue

[Cite any related issue(s) this pull request addresses. If none, simply state “None”]

Fixes: #(issue no.)

Description

[Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.]

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Screenshots / videos (if applicable)

[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]

Checklist:

  • I have made this change from my own.
  • I have taken help from some online resources.
  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers and screenshots after making the changes.

Copy link

vercel bot commented Oct 25, 2024

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

Name Status Preview Comments Updated (UTC)
buddy-trail ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 30, 2024 1:03pm

Copy link
Contributor

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our contributing.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

Copy link

netlify bot commented Oct 25, 2024

Deploy Preview for buddytrail ready!

Name Link
🔨 Latest commit db3a83f
🔍 Latest deploy log https://app.netlify.com/sites/buddytrail/deploys/67222e8980709b0008863b1c
😎 Deploy Preview https://deploy-preview-1490--buddytrail.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.

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Please Review My Changes and Merge it by Assigning Labels to It !!!
( Mam, Please Don't Close the Pull Request if it Shows RESOLVE CONFLICTS, Instead Reply ME in Such Case and Give Me Few Hours of Time in case I haven't Seen it, for Resolving Conflicts, and I will Reply Back when resolved conflicts and then Merge it, but Please don't close it. )

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006 Please see the conflicts . Resolve the conflicts .

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Conflicts are Resolved ! Thank You Mam, for Your Patience !

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006 Please see the conflicts .

Copy link

vercel bot commented Oct 26, 2024

Deployment failed with the following error:

Resource is limited - try again in 1 minute (more than 100, code: "api-deployments-free-per-day").

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Conflicts are Resolved !!!

Copy link

vercel bot commented Oct 26, 2024

Deployment failed with the following error:

Resource is limited - try again in 35 minutes (more than 100, code: "api-deployments-free-per-day").

Copy link

vercel bot commented Oct 26, 2024

Deployment failed with the following error:

Resource is limited - try again in 8 minutes (more than 100, code: "api-deployments-free-per-day").

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Conflicts have been Resolved ( Sorry Mam, for the Inconvinence, Because so many files were changed so it happening that that shows me 3-4 files Resolve Conflicts when I do it and You try to Again Merge it again shows me another 4-5 files for resolving Conflicts and not all at a time but once all file will get conflicts resolved you will be able to merge so please keep trying to merge so that it keep showing me resolve conflicts, Otherwise it keeps showing me no conflicts in merging the file, Hope you Understand the Problem and try to cooperate with it )

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006 Please look into conflicts
image

@PatelHarsh2006
Copy link
Contributor Author

PatelHarsh2006 commented Oct 27, 2024

@PriyaGhosal Finally Mam, the Conflicts are resolved and ready to Merge

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Please merge it by Assigning Labels to it !

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006
image

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Conflicts are resolved

@PriyaGhosal
Copy link
Owner

PriyaGhosal commented Oct 29, 2024

@PatelHarsh2006 In main page cursor is not working other pages its working
If unable to put tricolour to front page then add same sky blue to all pages as it should look uniform throughout the website ....
Please do it Harsh as website is not looking uniform
image

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Mam Can U Please Show Me Video what's Exactly the Problem is !

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006 Cursor is not uniform as main page will have sky cursor colour and other would have tricolour cursor

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Yes this Sky Color was before(my new changes) thats why it might be showing ( while other pages were not having any effect ), as You have not merge it yet. Mam, You can See in my video ( uploaded at the time of PR (👆)) index file also have tricolor effect only.

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006 In main page tricolour cursor is not showing

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006 Are you working on it

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Yes Mam, I am Working on it !

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Mam, Please Check If It is Working Now or Not ?

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006 no in main page tricolour cursor is not working

@PatelHarsh2006
Copy link
Contributor Author

Mam, Please Check Now !

@PriyaGhosal
Copy link
Owner

@PatelHarsh2006 it is showing conflicts

@PatelHarsh2006
Copy link
Contributor Author

@PriyaGhosal Conflicts have been resolved

@PriyaGhosal PriyaGhosal merged commit 09150a2 into PriyaGhosal:main Oct 31, 2024
5 checks passed
@PriyaGhosal
Copy link
Owner

@PatelHarsh2006
Great contribution ! 🎉 Your pull request has been successfully merged! Also Take a moment to ⭐ Star the repository if you haven't already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants