Skip to content
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

Add Africa, Africa Modest, Asia, Latin America, and Oceania character sets #8

Closed
Luisav1 opened this issue Nov 20, 2023 · 6 comments
Closed

Comments

@Luisav1
Copy link
Contributor

Luisav1 commented Nov 20, 2023

First need to add a preference model and then we'll add the classes and structure for the character sets.

@Luisav1 Luisav1 self-assigned this Nov 20, 2023
@amanda-phet amanda-phet changed the title Add Africa, Africa Modest, Asia, and Latin America character sets Add Africa, Africa Modest, Asia, Latin America, and Oceania character sets Nov 21, 2023
Luisav1 added a commit to phetsims/area-model-algebra that referenced this issue Nov 22, 2023
amanda-phet added a commit to phetsims/area-model-common that referenced this issue Nov 30, 2023
amanda-phet added a commit to phetsims/area-model-common that referenced this issue Nov 30, 2023
@amanda-phet amanda-phet self-assigned this Dec 1, 2023
Luisav1 added a commit to phetsims/area-model-common that referenced this issue Dec 1, 2023
Luisav1 added a commit that referenced this issue Dec 1, 2023
amanda-phet added a commit that referenced this issue Dec 1, 2023
amanda-phet added a commit that referenced this issue Dec 1, 2023
amanda-phet added a commit that referenced this issue Dec 1, 2023
amanda-phet added a commit that referenced this issue Dec 1, 2023
amanda-phet added a commit that referenced this issue Dec 1, 2023
amanda-phet added a commit that referenced this issue Dec 1, 2023
@amanda-phet
Copy link
Contributor

Artwork is all uploaded. Over to @Luisav1

@amanda-phet amanda-phet removed their assignment Dec 1, 2023
Luisav1 added a commit that referenced this issue Dec 5, 2023
…sets. Pass in character sets and jugglerController (character set controller) to the Multiplication game screen GenericGameScreen.js. See #8.
Luisav1 added a commit to phetsims/area-model-common that referenced this issue Dec 5, 2023
…sets. Pass in character sets and jugglerController (character set controller) to the Multiplication game screen GenericGameScreen.js. See phetsims/area-model-multiplication#8.
@Luisav1
Copy link
Contributor Author

Luisav1 commented Dec 5, 2023

The artwork is all integrated now. @amanda-phet could you review to ensure it's all good? Thanks!

@amanda-phet
Copy link
Contributor

The icons looked good, but now we're replacing them with the svg images. Let me know when those are ready for review.

@amanda-phet amanda-phet assigned Luisav1 and unassigned amanda-phet Dec 8, 2023
amanda-phet added a commit that referenced this issue Dec 12, 2023
amanda-phet added a commit that referenced this issue Dec 12, 2023
amanda-phet added a commit that referenced this issue Dec 12, 2023
amanda-phet added a commit that referenced this issue Dec 12, 2023
amanda-phet added a commit that referenced this issue Dec 12, 2023
amanda-phet added a commit that referenced this issue Dec 12, 2023
@amanda-phet
Copy link
Contributor

SVG screen icons are now uploaded.

@Luisav1
Copy link
Contributor Author

Luisav1 commented Dec 13, 2023

All the artwork is now replaced with the svg files. @amanda-phet sending this your way for your review.

@Luisav1 Luisav1 assigned amanda-phet and unassigned Luisav1 Dec 13, 2023
@amanda-phet
Copy link
Contributor

All of the artwork looks good to me!

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

No branches or pull requests

2 participants