Skip to content

Feat/uepr 164 merge scratch desktop into develop #238

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

Open
wants to merge 162 commits into
base: feat/uepr-165-merge-native-into-develop-with-platform
Choose a base branch
from

Conversation

KManolov3
Copy link
Contributor

@KManolov3 KManolov3 commented Apr 15, 2025

Resolves

https://scratchfoundation.atlassian.net/browse/UEPR-164

Proposed Changes

Merge changes from the branch containing merged native and develop branches. Conditionalize logic where necessary to preserve behaviour on the three platforms.

Reason for Changes

Another step towards the end goal of having a single published package, from which all three platform specific apps - www, scratch-desktop and native - use GUI.

Christopher Willis-Ford and others added 30 commits March 14, 2019 11:51
At some point these things will be changed / disabled / removed in
master or develop; at that point this change should be replaced with the
more official approach.
…ages-through-storage"

This brings "Load library images through storage" back into the
`scratch-desktop` branch since it's necessary here. We need to figure
out some performance details before trying to merge it into `develop`
though.
Go back to `scratch-storage` version 1.3.1 in the `scratch-desktop`
branch so that we have the performance improvements from
scratchfoundation/scratch-storage#83

If I understand correctly we reverted from 1.3.1 to 1.2.2 because of
problems affecting HTTP saves, which are not available in Scratch
Desktop.

This reverts commit b5a9c18, reversing
changes made to e91c32a.
[hotfix] Add Marian sprite and costumes
This is necessary for the current scratch-desktop build process but when
we merge this branch into develop we should revert this change.
`getItemIcons` (formerly `getItemImageSource`) now handles the current
library formats as well as upcoming library changes in which library
items will no longer have a `json` property. Or, at least, it should
work...

As part of that effort, `getItemIcons` now handles library items with
single or multiple icons internally, meaning that the caller no longer
needs to do extra work to check. Likewise, the `LibraryItem` container
now has just one `iconSource` property instead of an `icon` property for
the one-icon case and an `icons` property for the multi-icon case. The
LibraryItem component (not container) still only takes one icon, and the
container is still responsible for managing icon rotation.
fix costume animation in sprite library
KManolov3 and others added 27 commits April 8, 2025 13:04
feat: [UEPR-176] add logic for triggering feedback from editor
…-affects-editor

[IPR-350] Added no-global-reset to modal component
@KManolov3 KManolov3 requested a review from a team as a code owner April 15, 2025 13:23
@KManolov3 KManolov3 changed the base branch from develop to feat/uepr-165-merge-native-into-develop-with-platform April 15, 2025 13:23
Copy link


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


3 out of 7 committers have signed the CLA.
✅ (cwillisf)[https://github.com/cwillisf]
✅ (MiroslavDionisiev)[https://github.com/MiroslavDionisiev]
✅ (KManolov3)[https://github.com/KManolov3]
@christopher Willis-Ford
@ericrosenbaum
@rschamp
@picklesrus
Christopher Willis-Ford seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

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.

8 participants