-
Notifications
You must be signed in to change notification settings - Fork 822
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[3851] - Implement the functionality to send changes of add-block
#3952
[3851] - Implement the functionality to send changes of add-block
#3952
Conversation
…rlabs#3924) * fix:resolves hamburger opening(sugarlabs#3914) * fix:fixes sugarlabs#3914(suggested changes done) * adjust size of grid element repositioning when using aux toolbar --------- Co-authored-by: Walter Bender <[email protected]>
Co-authored-by: anas2357 <[email protected]>
Bumps [braces](https://github.com/micromatch/braces) to 3.0.3 and updates ancestor dependency [gulp](https://github.com/gulpjs/gulp). These dependencies need to be updated together. Updates `braces` from 2.3.2 to 3.0.3 - [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md) - [Commits](https://github.com/micromatch/braces/commits/3.0.3) Updates `gulp` from 4.0.2 to 5.0.0 - [Release notes](https://github.com/gulpjs/gulp/releases) - [Changelog](https://github.com/gulpjs/gulp/blob/master/CHANGELOG.md) - [Commits](gulpjs/gulp@v4.0.2...v5.0.0) --- updated-dependencies: - dependency-name: braces dependency-type: indirect - dependency-name: gulp dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* add support for cents through semi-tone transposition * add 50 cents macro * add cents example * add output frequencies * fix issue with ratio cents * add a ratio-cents example
* use processPitch for all pitch types * Add support for cents (sugarlabs#3937) * add support for cents through semi-tone transposition * add 50 cents macro * add cents example * add output frequencies * fix issue with ratio cents * add a ratio-cents example * use processPitch for all pitch types * convert freq to note before scalar step
* Update README.md * Update dockerfile
js/blocks.js
Outdated
@@ -6525,6 +6544,10 @@ class Blocks { | |||
} | |||
} | |||
} | |||
|
|||
if(this.activity.collaboration.hasCollaborationStarted){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use a space between if and (
js/blocks.js
Outdated
|
||
/** Signal that a new block is created */ | ||
if (this.activity.collaboration.hasCollaborationStarted) { | ||
if(this.isLocalUpdate){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
js/blocks.js
Outdated
@@ -3066,6 +3068,14 @@ class Blocks { | |||
this.visible = true; | |||
}; | |||
|
|||
// Emit a message when a new block is created | |||
this.emitAddedBlock = () => { | |||
if(this.activity.collaboration.hasCollaborationStarted){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here
add-block
Addresses: #3851
This PR implements the functionality to send
add-new-block
changes and apply those changes to the connected client's browsers.2024-07-16.16-55-01.mp4
Note: I have added a temporary
Collaborate
button in the toolbar to handle the infinite emitting of the events. later on, I'll shift this button to the planet.In order to test this on your local machine,
Collaborate
button from theToolbar