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

GridMapEditorPlugin: set_selected_palette_item does not update the cursor mesh #100721

Open
spoxii opened this issue Dec 22, 2024 · 0 comments · May be fixed by #100723
Open

GridMapEditorPlugin: set_selected_palette_item does not update the cursor mesh #100721

spoxii opened this issue Dec 22, 2024 · 0 comments · May be fixed by #100723

Comments

@spoxii
Copy link
Contributor

spoxii commented Dec 22, 2024

Tested versions

  • Reproducible since 4.4.dev [baf03e4]

System information

Godot v4.4.dev7 - Windows 11 (build 22631) - Multi-window, 1 monitor - Vulkan (Mobile) - dedicated AMD Radeon RX 6700 XT (Advanced Micro Devices, Inc.; 32.0.12011.1036) - AMD Ryzen 5 5600X 6-Core Processor (12 threads)

Issue description

GH-99639 exposed the GridMapEditorPlugin class to scripts.

However, the set_selected_palette_item function from this class does not update the cursor mesh as it would when selecting from the MeshLibrary palette.
The palette selection updates correctly and you will paint with the correct mesh item, but the cursor will still show the old mesh.

Steps to reproduce

Godot_v4.4-dev7_win64_B7kcDDyHop.mp4
  1. Open the MRP
  2. Make sure that the GridMapTool plugin is enabled
  3. Select the GridMap node in the scene tree
  4. In the GridMap bottom panel, select the paint tool so you can see the cursor mesh
  5. Open the new "GridMap Tool" panel in the bottom left dock
  6. Click a button ("red", "green", "blue") to select a mesh - this will call set_selected_palette_item
  7. Compare your choice with the current GridMap cursor.
    It should be the same color as your selection in the tool panel.

Minimal reproduction project (MRP)

godot-gridmap-cursor.zip

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

Successfully merging a pull request may close this issue.

2 participants