unify quicksand? #2429

Closed
opened 2022-08-08 22:01:58 +00:00 by flux · 4 comments
Member

do we need to keep both ethereal:quicksand and ethereal:quicksand2? the differences are that:

  1. ethereal:quicksand does not spawn in nature, cannot be crafted, looks exactly like normal sand, and if you are inside of it, you see pitch black
  2. ethereal:quicksand2 spawns, can be crafted, and if you are inside of it, has a slight blue tint vs. normal sand, and if you are inside it, your screen gets a heavy sand-colored overlay that you can see through slightly.
do we need to keep both `ethereal:quicksand` and `ethereal:quicksand2`? the differences are that: 1. `ethereal:quicksand` does not spawn in nature, cannot be crafted, looks exactly like normal sand, and if you are inside of it, you see pitch black 2. `ethereal:quicksand2` spawns, can be crafted, and if you are inside of it, has a slight blue tint vs. normal sand, and if you are inside it, your screen gets a heavy sand-colored overlay that you can see through slightly.
flux added the
4. step/question
1. kind/balancing
labels 2022-08-08 22:01:58 +00:00
AliasAlreadyTaken was assigned by flux 2022-08-08 22:01:59 +00:00
flux added the
4. step/ready to QA test
label 2022-11-03 13:33:37 +00:00
Author
Member

also fixed in ad306ff6ad.

solved by getting rid of the "old style" quicksand which didn't have any specific use.

also fixed in https://gitea.your-land.de/your-land/yl_commons/commit/ad306ff6ad38ecededb886ab74a31405f0580be7. solved by getting rid of the "old style" quicksand which didn't have any specific use.
AliasAlreadyTaken added this to the 1.1.116 milestone 2022-11-03 16:28:10 +00:00

I remember either you or someone else once looked into the same problem and found there was some reason why it was done that way. As far as my memory goes, we found we wanted to unify it back then as well.

Or I confuse this situation with a similar one.

I remember either you or someone else once looked into the same problem and found there was some reason why it was done that way. As far as my memory goes, we found we wanted to unify it back then as well. Or I confuse this situation with a similar one.
Author
Member

the previous issue was #2163, about quicksand not falling. the documented difference was very minor.

the previous issue was #2163, about quicksand not falling. the documented difference was very minor.
flux added
5. result/fixed
and removed
4. step/question
4. step/ready to QA test
labels 2022-11-13 21:55:38 +00:00
AliasAlreadyTaken was unassigned by flux 2022-11-13 21:59:59 +00:00
Author
Member

this is live

this is live
flux closed this issue 2022-11-13 22:00:05 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: your-land/bugtracker#2429
No description provided.