Skip to content

Update Bounce Animation on AppCard.vue#127

Open
malikobansa wants to merge 1 commit intoSovereinia:mainfrom
malikobansa:main
Open

Update Bounce Animation on AppCard.vue#127
malikobansa wants to merge 1 commit intoSovereinia:mainfrom
malikobansa:main

Conversation

@malikobansa
Copy link

@malikobansa malikobansa commented Jul 24, 2025

Description

What does this PR do? What problem does it solve?

Changes made

  • Feature added
  • Bugfix
  • Refactoring
  • Docs update
  • Other (please describe):

Related issue

Closes #

How to test it

Briefly describe how someone can test or validate this change.

Checklist

  • My code follows the style of this project
  • I’ve tested it or verified it works as expected
  • I’ve added comments or documentation where needed

Contributor Statement

By submitting this pull request, I confirm that:

  • I am voluntarily contributing this code under the project's license
  • I have the legal right to do so
  • I understand this contribution may be modified or redistributed under that license

@vinybk
Copy link
Contributor

vinybk commented Jul 25, 2025

Hey @malikobansa Thank you for your contribution. Will review it in two days

@vinybk
Copy link
Contributor

vinybk commented Jul 31, 2025

Hi @malikobansa
The jump effect looks good. However, I think it would give a more clear indications to users if only one of the cards jumped instead of all of them. It could be the first or second one.

Can you make that adjustment before we merge?

@malikobansa
Copy link
Author

Yes sir

@malikobansa
Copy link
Author

@vinybk sir my windows just crashed sorry for the inconvience

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.

2 participants