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

Fixed "doubleclicking" on Colorpicker #363

Merged
merged 3 commits into from
Nov 11, 2024
Merged

Conversation

Paalui
Copy link
Contributor

@Paalui Paalui commented Sep 11, 2024

Bug: #315 / #345 / #309
Didn't read all the code so hope i got it right.

When you close a colorchannel it saves the atoms to reuse the gradiets. When opening it again it checks if the old atoms are there and it copies the gradient. But it only checks if the atom is there so when you close the channel before it has drawn all gradients, the gradient is undefined but isGradient is true.

I made it that it also copies the isGradient value from the old atom instead of just setting it true.

Bug: TodePond#315 / TodePond#345 / TodePond#309
Didn't read all the code so hope i got it right.

When you close a colorchannel it saves the atoms to reuse the gradiets. When opening it again it checks if the old atoms are there and it copies the gradient. But it only checks if the atom is there so when you close the channel before it has drawn all gradients, the gradient is undefined but isGradient is true.

I made it that it also copies the isGradient value from the old atom instead of just setting it true.
Copy link

netlify bot commented Sep 11, 2024

Deploy Preview for keen-boyd-1443aa ready!

Name Link
🔨 Latest commit c1818a2
🔍 Latest deploy log https://app.netlify.com/sites/keen-boyd-1443aa/deploys/66ea1a2f56da5200083f9cc8
😎 Deploy Preview https://deploy-preview-363--keen-boyd-1443aa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

set dx and dy to 0 for the missing cases, when a square gets dropped in a paddle.
If you make the UI too small it get broken and you have to reset.
@Paalui
Copy link
Contributor Author

Paalui commented Sep 18, 2024

Sorry to interrupt.
I just found out that my new commits have ended up here.
I also fixed #316 / #334 / #336
and limited zoomig out the UI

@TodePond TodePond merged commit 8ed7035 into TodePond:main Nov 11, 2024
4 checks passed
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.

2 participants