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.

Sign up for GitHub

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

Jump to bottom

Cursor loses its texture when leaving the inventory #159

Closed
mapedorr opened this issue Feb 24, 2024 · 1 comment
Closed

Cursor loses its texture when leaving the inventory #159

mapedorr opened this issue Feb 24, 2024 · 1 comment
Assignees
@mapedorr
Labels
9 verbs Specific for the 9 Verbs GUI bug Something isn't working

Comments

@mapedorr
Copy link
Collaborator

mapedorr commented Feb 24, 2024 •
edited

Bug description

When an inventory item is clicked (in the 9 Verbs interface) the cursor uses the texture of the item, but as soon as it leaves the button the texture gets hidden.

Steps to reproduce

You HAVE to activate the 9 Verbs interface:

  1. Put an item in the inventory.
  2. Click on it to make it the active inventory item. The texture of the item will appear on top of the cursor's default texture.
  3. Move the cursor out of the inventory.
  4. The cursor will keep its default texture and will lose the texture of the selected item.

Expected vs observed behavior

Both in the classic (old LucasArts games) and modern (Thimbleweed Park) 9 Verbs interface when an inventory item is clicked, the cursor doesn't take the shape of the item. It only shows the info of the "Use xxx in yyy" in the hover text.

Environment information (please complete):

  • OS type and version: Windows 11
  • Godot version: 4.2.1
  • Plugin version or branch: beta 1

Additional context

Nothing.
@mapedorr mapedorr added bug Something isn't working 9 verbs Specific for the 9 Verbs GUI labels Feb 24, 2024
@mapedorr mapedorr self-assigned this Feb 24, 2024
mapedorr added a commit that referenced this issue Feb 24, 2024
@mapedorr
Fixes for #159 and #160
a8a4c82 
Improve behavior of all GUI templates for items use.
[upd] Use Mono font for Label in GUI copy progress popup.
[fix] Validate existance of avatars to fix Portrait component.
[fix] Now item use fallback is handled by the commands script of each
GUI template.
@mapedorr
Copy link
Collaborator Author

Fixed in #163

mapedorr added a commit that referenced this issue Feb 25, 2024
@mapedorr @stickgrinder
Fixes for #159 #160 #158 #162 (#163)
fc9a89e 
* Fixes for #159 and #160

Improve behavior of all GUI templates for items use.
[upd] Use Mono font for Label in GUI copy progress popup.
[fix] Validate existance of avatars to fix Portrait component.
[fix] Now item use fallback is handled by the commands script of each
GUI template.

* Fix #158

Ignore the polygon assignation process when editing a PopochiuClickable
from its .tscn file, or when it is a PopochiuCharacter.

* Fix #162

Now the copy process text is shown inside the Setup popup.

[fea] Also adds an Output message to inform the copy process is
completed.
[upd] Style of slots in 9 Verb GUI wasn't correct.
[fix] Update script templates so the method _on_item_used calls
E.command_fallback().

* Update message with potential misunderstanding

Cambridge dictionary says that: an assignation is a meeting that is secret or not allowed, especially one between two people having a romantic relationship.

Co-authored-by: Paolo Pustorino <stickgrinder@gmail.com>

* Use a progress bar to show GUI template process

Update code to follow suggestions made by @stickgrinder .

* Update style of copy progress panel to match the engine theme

---------

Co-authored-by: Paolo Pustorino <stickgrinder@gmail.com>
@mapedorr mapedorr mentioned this issue Mar 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mapedorr mapedorr

Labels
9 verbs Specific for the 9 Verbs GUI bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mapedorr