JeCel reports: Maybe add a button with which ... #2323
Labels
No Label
1. kind/balancing
1. kind/breaking
1. kind/bug
1. kind/construction
1. kind/documentation
1. kind/enhancement
1. kind/griefing
1. kind/invalid
1. kind/meme
1. kind/node limit
1. kind/other
1. kind/protocol
2. prio/controversial
2. prio/critical
2. prio/elevated
2. prio/good first issue
2. prio/interesting
2. prio/low
3. source/art
3. source/client
3. source/engine
3. source/ingame
3. source/integration
3. source/lag
3. source/license
3. source/mod upstream
3. source/unknown
3. source/website
4. step/approved
4. step/at work
4. step/blocked
4. step/discussion
4. step/help wanted
4. step/needs confirmation
4. step/partially fixed
4. step/question
4. step/ready to deploy
4. step/ready to QA test
4. step/want approval
5. result/cannot reproduce
5. result/duplicate
5. result/fixed
5. result/maybe
5. result/wontfix
ugh/petz
ugh/QA main
ugh/QA NOK
ugh/QA OK
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: your-land/bugtracker#2323
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
JeCel reports a bug:
Player position:
Player look:
Player information:
Player meta:
Log identifier
Profiler save:
Status:
Teleport command:
Compass command:
Would it be possible to get a refund? :(
@JeCel the issue w/ dragging wrong items into the trash is due to the fact that minetest registers a click event both when you press the mouse button and when you release it. if you're moving the cursor while clicking on something, it can end up clicking once on one item, and then clicking a second time on another.
i created an upstream issue, because while i've known about this for years, apparently no-one else ever thought about reporting it.
https://github.com/minetest/minetest/issues/12596
adding an "undo" feature to the inventory trashcan has been on my todo list for years, that might be easier...
I just realized that I wrote potion. Of course I meant button...
@flux Ah, that definitely explains that behavior. It's really frustrating
upstream issue:
https://github.com/minetest-mods/unified_inventory/issues/186
Now I want to implement a wayback-potion :D
Maybe we should throw out the trashcan. So far it lead to more trouble than good. If people really want to throw away things, they can simply drag them out of their inventory and wait for them to despawn.
This kind of includes a way back: You can anytime pick up what you threw out again.
Another benefit: There's already a setting to disable the trashcan.
unified_inventory_trash (Enable trash) bool true
worth trying. i guarantee that players will complain about that too, but if they're really upset, we can just re-enable it.
we could also create a trash-can node. there seems to be one in this mod https://content.minetest.net/packages/Evergreen/trash_can/ but we'd have to disable the dumpster.
JeCel reports: Maybe add an potion with which ...to JeCel reports: Maybe add a button with which ...We disabled it, way more players than I could have imagined complained, so we reenabled it until #4077 is ready.
The suggested trashcan mod looks funny, but has the same missclick opportunity.
Let's have this issue deal with the client missclick and explore the trashcan in #4077
closing this as resolved, since we have #4077 now. for reference, the usptream client/engine issue is resolved and should be part of 5.8