Skip to content

Techdebt/update core#115

Merged
kcw-grunt merged 9 commits into
developfrom
techdebt/update-core
Jun 7, 2026
Merged

Techdebt/update core#115
kcw-grunt merged 9 commits into
developfrom
techdebt/update-core

Conversation

@kcw-grunt

@kcw-grunt kcw-grunt commented Jun 7, 2026

Copy link
Copy Markdown
Collaborator

📱 Description

This PR adds audio feedback to the balance visibility toggle in the BalanceBentoView. When users tap the eye icon to show/hide their balance, a playful "boing" sound effect now plays, improving the tactile feedback and user experience of the app.

Platform

  • iOS

🎯 Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🔧 Refactoring (code change that neither fixes a bug nor adds a feature)
  • 📚 Documentation update
  • 🎨 UI/UX improvement
  • ⚡ Performance improvement
  • 🧪 Test addition or improvement
  • ⚙️ CI/CD change

📋 Changes

New Components Added

  • boingspringmouthharp042013.mp3 – Sound asset for balance toggle audio feedback

Modifications

  • BalanceBentoView.swift – Added playBoing() private method that triggers sound playback via SoundsHelper; integrated sound playback into the balance visibility toggle button action
  • Modules/core – Updated core module (details not visible in diff)
  • brainwallet.xcodeproj/project.pbxproj – Updated project configuration to reference new sound asset

Removals

  • None

📊 Statistics

  • Additions: 10 lines
  • Deletions: 1 lines
  • Files Changed: 4
  • Commits: 2

@kcw-grunt
kcw-grunt requested a review from amobileagent-gs June 7, 2026 20:46
@kcw-grunt kcw-grunt self-assigned this Jun 7, 2026
@kcw-grunt kcw-grunt added the enhancement New feature or request label Jun 7, 2026
@gruntsoftware gruntsoftware deleted a comment from github-actions Bot Jun 7, 2026
@kcw-grunt
kcw-grunt merged commit be83d7b into develop Jun 7, 2026
1 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant