-
Notifications
You must be signed in to change notification settings - Fork 364
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
April Fools 2024 easter egg: Scratch Haddocks #7299
April Fools 2024 easter egg: Scratch Haddocks #7299
Conversation
libraries/common/global-theme.js
Outdated
const now = Date.now() / 1000; | ||
// TODO: update these timestamps for year 2024 | ||
if (/*now < 1648911600 && now > 1648738800 && */ ["popup"].includes(location.pathname.split("/")[2])) { | ||
document.documentElement.style.setProperty("--brand-orange", "#3232fd"); |
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.
This is not the same blue as the prerelease versions
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.
This is not the same blue as the prerelease versions
Sorry if this is a dumb question that may be a bit off-topic, but, why did you say that in a review on your own PR while you could've just put that in a comment?
Timestamps for this year are |
Is there anything else we could replace with emojis of sea creatures? 😁 Maybe some of the icons? Or could we just have haddocks swimming/floating around on the popup? |
Made it: Scratch.Addons.but.it.s.a.fish.tank.mp4These are the fish emoji from Windows 11 (the OS I'm using), BTW |
Code for that available at https://github.com/DNin01/ScratchAddons/tree/afd24-fish-tank @WorldLanguages You can add it to this PR if you want. |
lol! I don't know what to think |
I'm not sure if those are 3 hours off or not? Time zones make it confusing |
Why use timestamps? var currentDate = new Date();
var currentMonth = currentDate.getMonth();
var currentDay = currentDate.getDate();
if ((currentMonth === 2 && currentDay >= 31) || (currentMonth === 3 && currentDay <= 2)) {
// The code to do when the date is march 31st - April 2nd here.
} Can do the trick.. |
I think we want it to appear at the same time for all users. Timestamps ignore time zones, your code doesn't. |
This comment was marked as resolved.
This comment was marked as resolved.
Months start from 0 in js if you're confused. |
And days don't?! |
Welcome to JavaScript! |
Should we add those then? As long as they aren't that annoying, it should be fine... EDIT: I added the swimming fish to this PR. |
(We wouldn't show the fish animations if reduced-motion was toggled by the user) |
I propose that we also temporarily replace the dangos with haddocks because it's easy and why not? |
Replace which dangos? |
If we could collect some projects related to haddocks, we could feature them in a separate section under the featured projects section in the homepage like featured dangos? |
Probably in |
|
What shade of blue is best to represent fish? EDIT: I changed it to "Bluetiful" |
Aqua blue in my opinion. |
Okay, the haddocks shouldn't swim through the edges anymore. |
Are we still planning to merge this and release 1.37, and expecting CWS to review this before Monday? There's not much time... |
Looks ready to merge @DNin01 @Samq64 @mybearworld Any approvals? To test it you can replace the 4 instances of |
I also tested this with reduced motion enabled. |
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.
The logo is'nt changed yet... |
I might be wrong on this, but changing the logo would probably delay the process of getting this approved on the web stores further. |
Also, changing the logo dynamically only on April Fools sounds like a headache... |
By logo i only meant on the popup. See WorldLanguages#5 |
Reviews within 1 hour please |
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.
Tested on Firefox
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.
LGTM
That probably should've been implemented - these haddocks are actually pretty annoying when trying to write a comment. |
Yup... The good things are:
|
This reverts commit 643fca2.
Right now only the top part of the popup has any changes at all
(THIS SCREENSHOT IS NOW OUTDATED)
![image](https://private-user-images.githubusercontent.com/17484114/316617376-3f6b34a7-9b02-4dd8-9017-6fcb923fba16.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA0NjQ0MzIsIm5iZiI6MTcyMDQ2NDEzMiwicGF0aCI6Ii8xNzQ4NDExNC8zMTY2MTczNzYtM2Y2YjM0YTctOWIwMi00ZGQ4LTkwMTctNmZjYjkyM2ZiYTE2LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA4VDE4NDIxMlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWI4OTU1YWFiZDRkMTQ3MWViM2U0NzU4M2JmMTBhOGRjZWEzYWVkMGYyMGQ4NDI3MWJmZjVhNzk0Y2FhYzNjZTYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.gC9f_o31DNmC8sdmJBkppdcN_Ae47kgd7S0j9OYC178)