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

Staple scarves to scarves in the scarf stapler #3

Closed
sisby-folk opened this issue Sep 9, 2022 · 5 comments
Closed

Staple scarves to scarves in the scarf stapler #3

sisby-folk opened this issue Sep 9, 2022 · 5 comments

Comments

@sisby-folk
Copy link

sisby-folk commented Sep 9, 2022

Hey hi! I think it'd be super useful if you could put two scaves in the stapler, and the second scarf's colours would "append" to the first scarf on each side.

This would mean that you could instantly make a pridelib creative scarf twice as long, just by stapling it to itself.

In turn I think it would be super nice if the pridelib scarves were shorter by default (they're 5x right now I think?) I'm partial to 1x, but obvs that's your call

though 1x would mean players can quickly make and wear an aro-ace-nonbinary scarf, which, sounds great.

@falkreon
Copy link
Owner

falkreon commented Sep 9, 2022

Pros:

  • Easy to implement
  • Lots of time saved for creative mode and semi-creative scenarios
  • Scarves definitely need easier mix-and-match for two or three or eight flags
  • Long, long scarves

Cons:

  • Shift-clicking no longer really tells you where the scarf goes in the UI. Potential survival mode slowdown and/or confusion
  • If someone was really trying to, all these speedups might actually, finally allow them to crash the server with an unsendably large NBT packet. Not sure I want to risk it for the convention
  • All currently extant scarves would stay the exact same way that they are because it's all NBT driven and their NBT will stay the same.

I could mitigate this first con by making sure the scarf slot is slot 0. I just checked, and it's slot zero. Whoops, I accidentally a QoL feature.

Third con is also kind of a pro. I like not forcing people to change out their scarves.

@sisby-folk
Copy link
Author

sisby-folk commented Sep 9, 2022

I had some other confusions with the scarf stapler i had to remedy myself, and I think the solution is to have a scarf outline in the right slot. that's the base scarf (like in smithing), then you have the ingredient slots that can stay blank.

@falkreon
Copy link
Owner

falkreon commented Sep 9, 2022

The correct, center slot you mean? Yes. I'll have to add outlines to the slots in order to clarify if this becomes a thing.

What should the behavior be if one staples a scarf with both tails set to e.g. the left side?

  • Stapling the left tail and ignoring the right is eating squares and I don't super like that
  • Stapling the left tail, then the right tail, to the left tail of the destination scarf seems surprising and I don't like surprise in my systems
  • Stapling the left tail to the left tail end, and the right tail to the right tail end ignores the fact that the additional scarf, the one that's being added, was placed in the left slot. Is that an acceptable compromise?

If stapling either side staples both sides of a two-tailed scarf, then is it surprising if: the user takes a single-(left-)tailed scarf and places it into the right side of the stapler to staple it to the end of the right side of a scarf, and it "moves" the data to the right side for stapling?

@sisby-folk
Copy link
Author

sisby-folk commented Sep 9, 2022 via email

@falkreon
Copy link
Owner

I hate this behavior, but I think you're right.

This would also stop the user cold before they mistakenly put their own scarf in the left slot and the scarf they want to staple to it in the middle slot, and staple them backwards. There's no undo for this.

I think rather than detecting it on the back-end and trying to communicate to the user why it failed, I want to proactively examine the NBT and deny scarves in the square slots which have a LeftSlot and RightSlot key.

Obviously we still need slot icons to guide the user.

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

No branches or pull requests

2 participants