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

Paste Value can empty an array depending on right-click location #80974

Closed
Calinou opened this issue Aug 24, 2023 · 1 comment · Fixed by #80977
Closed

Paste Value can empty an array depending on right-click location #80974

Calinou opened this issue Aug 24, 2023 · 1 comment · Fixed by #80977

Comments

@Calinou
Copy link
Member

Calinou commented Aug 24, 2023

Godot version

4.2.dev 6758a7f

System information

Godot v4.2.dev (6758a7f) - Fedora Linux 38 (KDE Plasma) - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4090 (nvidia; 535.98) - 13th Gen Intel(R) Core(TM) i9-13900K (32 Threads)

Issue description

Paste Value on an exported Array property can empty the array depending on right-click location:

simplescreenrecorder-2023-08-25_00.38.48.mp4

Steps to reproduce

  • Create a script with an exported array property.
  • Save the scene (and script) and select the node.
  • Add 3 items to the array with any type (int is used in this example).
  • Set non-default values for each item.
  • Right-click an item and choose Copy Value.
  • Aim to the left of the item's name (near the drag-and-drop reorder icon), right-click another item and choose Paste Value.
  • Notice how the array's size is reset to 0 and all values are lost (this can be undone).

Minimal reproduction project

test_array_paste_value_empty.zip

@ajreckof
Copy link
Member

I think this encompass two different issue.
First that you are able to call the popup menu of the whole array where you shouldn't. Second that pasting an int to an array will wipe out the array this seems probably legitimate since there is no way to convert an int but it does so while spouting a lot of errors and preventing you from using the PropertyEditor. The second part is strange as it is happening even when the property is typed and shouldn't be able to hold an int at all :(.

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.

3 participants