Skip to content

Fix LEGO Bricks play button icon toggle during playback#7461

Open
bharat-DeV07 wants to merge 1 commit into
sugarlabs:masterfrom
bharat-DeV07:fix-play-button-toggle
Open

Fix LEGO Bricks play button icon toggle during playback#7461
bharat-DeV07 wants to merge 1 commit into
sugarlabs:masterfrom
bharat-DeV07:fix-play-button-toggle

Conversation

@bharat-DeV07

@bharat-DeV07 bharat-DeV07 commented Jun 2, 2026

Copy link
Copy Markdown
  • Bug fix

Description

Fixed the LEGO Bricks play button icon toggle issue.

Change Made

  • Updated play button icon during playbacks
  • Reverted icon back after playback ends

Fixes #7455

@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

🧪 Jest Test Results

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Coverage: Statements: 48.22% | Branches: 39.77% | Functions: 52.94% | Lines: 48.62%
Master Coverage: Statements: 48.23% | Branches: 39.77% | Functions: 52.95% | Lines: 48.62%

Note: These failures may be introduced by this PR or may already exist in the master branch.
Tip: Update your branch with the latest master and rerun tests.
If the same failures are present on master, they are likely not introduced by this PR.

Failed Tests:

aidebugger.test.js

@bharat-DeV07

Copy link
Copy Markdown
Author

Implemented the requested fix and tested locally.
Please let me know if any further improvements are needed.

@github-actions github-actions Bot added bug fix Fixes a bug or incorrect behavior size/XS Extra small: < 10 lines changed area/javascript Changes to JS source files labels Jun 2, 2026

@lavjeetrai lavjeetrai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great initiative to improve the UX! Adding the visual feedback is a really nice touch.

However, it would be much better if the pause button actually worked to pause or stop the audio.

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

Labels

area/javascript Changes to JS source files bug fix Fixes a bug or incorrect behavior size/XS Extra small: < 10 lines changed

Projects

Development

Successfully merging this pull request may close these issues.

[Bug] LEGO Bricks play button icon does not toggle during playback

2 participants