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

11 add visual bing bong when an artist is added to a country #51

Merged

Conversation

mold
Copy link
Owner

@mold mold commented Jan 25, 2024

It a flash rather than a bing bong

@mold mold linked an issue Jan 25, 2024 that may be closed by this pull request
@mold mold force-pushed the 11-add-visual-bing-bong-when-an-artist-is-added-to-a-country branch 2 times, most recently from b8560e7 to 9c9d8be Compare January 27, 2024 20:00
mold added 4 commits January 27, 2024 21:02
Used for the css transform-origin, could also be used for zooming correctly?
 - Replace calls to `map.putCountryCount` with `map.addArtists`
 - Reduce logic in `putCountryCount` by  passing better data from `script.js`
 - flash a country when it gets a new artist!!! wow!
@mold mold force-pushed the 11-add-visual-bing-bong-when-an-artist-is-added-to-a-country branch from 9c9d8be to a9bca3b Compare January 27, 2024 20:04
@mold mold marked this pull request as ready for review January 27, 2024 20:07
@fymmot fymmot merged commit 5fd4a57 into master Jan 28, 2024
1 check passed
@fymmot fymmot deleted the 11-add-visual-bing-bong-when-an-artist-is-added-to-a-country branch January 31, 2024 22:58
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.

Add visual "bing, bong" when an artist is added to a country
2 participants