sugarlabs / musicblocks

Music Blocks -- A musical microworld
https://musicblocks.sugarlabs.org/
GNU Affero General Public License v3.0
539 stars 727 forks source link

Bugs in Temperament Block #3823

Closed falgun143 closed 3 months ago

falgun143 commented 3 months ago

Description

There are mutilple bugs in the present temperament block 1.When ever the time difference between two clicks on the play/stop button is less then a new playback is generated which results in collision of the music 2.When we click on the pause button the next note is also highlited.

  1. When the notes are played in reverse order and when we click on pause button the note still remain highlited .

Expected Behavior

1.When ever the time difference between two clicks on the play/stop button is less then either of the two should happen 1.1 If the note was still playing then it should resume from where it stopped. 1.2 And if the note is completed then it should again start from zero.

2.When we click on the pause button the next note should not get highlited.

  1. When the notes are played in reverse order and when we click on pause button the note should have the initial color .

Screenshots

These are the video's showing all the bugs

1. https://github.com/sugarlabs/musicblocks/assets/105922142/0e465a45-fed1-4dc8-b9c9-59568a9fe970

2. https://github.com/sugarlabs/musicblocks/assets/105922142/41694dd1-5aa2-4902-9267-55f29bd0a842 3.

https://github.com/sugarlabs/musicblocks/assets/105922142/e67cd5c7-d410-4667-b5f3-01a04cf2adad

How to Reproduce

Switch to advanced mode and select temperament.

Console log Errors:

Environment:

Checklist


Thank you for contributing to our project! We appreciate your help in improving it.

📚 See contributing instructions.

🙋🏾🙋🏼 Questions: Community Matrix Server.

pikurasa commented 3 months ago

These improvements all seem good to me. @falgun143 do you plan to work on them, or are you just reporting them? (Just reporting is fine, too)

falgun143 commented 3 months ago

@pikurasa I have almost done ,Will raise a pr soon.