-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Update Kanvas visuals and assets as per issue #6482 #7035
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
base: master
Are you sure you want to change the base?
Update Kanvas visuals and assets as per issue #6482 #7035
Conversation
Signed-off-by: AasthathecoderX <bhat.aasthaa@gmail.com>
@Namanv0509 ,
Clarification needed: Could the maintainers confirm if this approach is correct, and if you need the icons changed inside all animated card assets(Image-2) ? Thank you! |
🚀 Preview for commit cfe6a40 at: https://68edb4aa889a46c372d11de0--layer5.netlify.app |
THis is better! |
@AasthathecoderX kindly check. The linting check is failing |
Hi @FreemanBoss, src/components/Pricing/PricingAddons/index.js src/sections/Community/Handbook/recognition.js src/sections/Projects/Sistent/components/iconbutton/index.js However, the only file I changed in this PR is: src/sections/Kanvas/kanvas-catalog.js These lint errors seem unrelated to my changes. Please let me know if you’d like me to address these, or if this will be handled separately. Thanks |
@AasthathecoderX I'm actually not a maintainer rather a co contributor. I'd suggest you try pulling updates and probably it might have been fixed already. |
🚀 Preview for commit a65445f at: https://68f3812d7f664777b4b55dba--layer5.netlify.app |
Description
This PR fixes #6482
Notes for Reviewers
Signed commits