Skip to content

Fix LineEdit with secret checked reveals the secret when a selection is dragged#106565

Merged
Repiteo merged 1 commit into
godotengine:masterfrom
pippenpaddleopsicopolis:line-edit-secret-drag
May 19, 2025
Merged

Fix LineEdit with secret checked reveals the secret when a selection is dragged#106565
Repiteo merged 1 commit into
godotengine:masterfrom
pippenpaddleopsicopolis:line-edit-secret-drag

Conversation

@pippenpaddleopsicopolis
Copy link
Copy Markdown
Contributor

@pippenpaddleopsicopolis pippenpaddleopsicopolis commented May 18, 2025

Fixes #106564

Checked how an HTML5 <input type="password"> behaves and the drag and drop are disabled.

Before

Screen.Recording.2025-05-18.at.18.22.02.mov

After

Screen.Recording.2025-05-18.at.18.30.18.mov

@pippenpaddleopsicopolis pippenpaddleopsicopolis requested a review from a team as a code owner May 18, 2025 16:27
@KoBeWi
Copy link
Copy Markdown
Member

KoBeWi commented May 18, 2025

A property disabling another property? Not to mention that you can still enable dragging afterwards (or it can be enabled accidentally when loading a scene).

Instead you should disable dragging regardless of drag_and_drop_selection_enabled if secret is enabled.

@KoBeWi KoBeWi added this to the 4.5 milestone May 18, 2025
@pippenpaddleopsicopolis
Copy link
Copy Markdown
Contributor Author

A property disabling another property?

Still learning the ways of Godot :)

Instead you should disable dragging regardless of drag_and_drop_selection_enabled if secret is enabled.

Makes sense! Will update the PR.

@Repiteo Repiteo merged commit 3cfe5f9 into godotengine:master May 19, 2025
20 checks passed
@Repiteo
Copy link
Copy Markdown
Contributor

Repiteo commented May 19, 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.

LineEdit with secret checked reveals the secret when a selection is dragged

3 participants