X Tutup
Skip to content

Fix ColorPickerButton close popup on mouse click#98428

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
pafuent:fixing_color_picker_closing_popup_on_mouse_click
Nov 20, 2025
Merged

Fix ColorPickerButton close popup on mouse click#98428
Repiteo merged 1 commit intogodotengine:masterfrom
pafuent:fixing_color_picker_closing_popup_on_mouse_click

Conversation

@pafuent
Copy link
Contributor

@pafuent pafuent commented Oct 22, 2024

Fixes #91813

There is a proposal to fix this properly, because it is out of my league and it seems that nobody started to work on it, I decided to propose this fix as a temporary measure until that proposal is implemented.

ColorPickerBug ColorPickerFix

@pafuent pafuent requested a review from a team as a code owner October 22, 2024 17:58
@Chaosus Chaosus added this to the 4.4 milestone Oct 23, 2024
@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from 589b679 to 6290f1b Compare November 13, 2024 11:10
@pafuent
Copy link
Contributor Author

pafuent commented Nov 13, 2024

Can I get a review?

@KoBeWi
Copy link
Member

KoBeWi commented Nov 13, 2024

There is another fix for that issue (#93775), but this one is simpler, so it's more preferable.
However I left a comment in the issue: #91813 (comment)
If the other bug I mentioned is fixed, this will become irrelevant. I'm not sure if it's worth to fix it now.

@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from 6290f1b to b24d56f Compare November 19, 2024 02:52
@pafuent
Copy link
Contributor Author

pafuent commented Nov 19, 2024

(updating to the latest changes on master just because I was here 😉 )

@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from b24d56f to 32caf48 Compare November 29, 2024 16:39
@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from 32caf48 to 077a5fc Compare January 21, 2025 13:51
@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from 077a5fc to 612879f Compare February 6, 2025 21:52
@Repiteo Repiteo modified the milestones: 4.4, 4.5 Feb 24, 2025
@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from 612879f to 6b29011 Compare March 5, 2025 15:00
@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from 6b29011 to b16b488 Compare March 19, 2025 11:30
@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from b16b488 to 9e7765c Compare April 11, 2025 02:29
@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from 9e7765c to c82ce1f Compare May 1, 2025 00:14
@Repiteo Repiteo modified the milestones: 4.5, 4.6 Sep 8, 2025
@pafuent pafuent force-pushed the fixing_color_picker_closing_popup_on_mouse_click branch from c82ce1f to 2ef3ebf Compare September 18, 2025 11:57
@Repiteo Repiteo requested a review from KoBeWi November 18, 2025 20:25
Copy link
Member

@KoBeWi KoBeWi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My comment from a year ago is still valid, but I guess the fix for underlying issue is not coming anytime soon 🤷‍♂️

@Repiteo Repiteo merged commit b79fe2e into godotengine:master Nov 20, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 20, 2025

Thanks!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ColorPickerButton toggle mode has no effect on color picker visibility

4 participants

X Tutup