Skip to content

adding class name to popped ballons, removed unnecessary func, title change - #1034

Merged
swapnilsparsh merged 1 commit into
swapnilsparsh:masterfrom
Marin303:balloons
May 25, 2023
Merged

swapnilsparsh merged 1 commit into
swapnilsparsh:masterfrom
Marin303:balloons

Conversation

@Marin303

Copy link
Copy Markdown
Contributor

🛠️ Fixes Issue (#999 )

👨‍💻 Changes proposed and Brief Description

  • adding class name to already popped ballons
  • removed unnecessary function > removeEvent
  • title spelling fixed

✅ Check List (Check all the applicable boxes)

  • ✅ My code doesn't break any part of the project
  • ✅ This PR does not contain plagiarized content.
  • ✅ My Addition/Changes works properly and matches the overall repo pattern.
  • ✅ The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

balloons

@netlify

netlify Bot commented May 24, 2023

Copy link
Copy Markdown

Deploy Preview for 30daysofjs ready!

Name Link
🔨 Latest commit cc4a571
🔍 Latest deploy log https://app.netlify.com/sites/30daysofjs/deploys/646e29f9152dde0008b53825
😎 Deploy Preview [https://deploy-preview-1034--30daysofjs.netlify.app/31 - pop the balloons](https://deploy-preview-1034--30daysofjs.netlify.app/31 - pop the balloons)
📱 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 settings.

@swapnilsparsh
swapnilsparsh merged commit 6e49d5e into swapnilsparsh:master May 25, 2023
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