LiquidBounce icon indicating copy to clipboard operation
LiquidBounce copied to clipboard
verified publisher icon CCBlueX

feat: breadcrumbs rework

Open ccetl opened this issue 1 year ago • 2 comments

  • breadcrumbs are time based
  • optional trails for other players
  • the height as setting & a fade effect (#2567)

ccetl avatar May 26 '24 14:05 ccetl

I'm generally not that happy with the implementation. I probably will restructure it to be less messy but not sure since I don't want to make this a multi file module as this isn't really the style in this client for minor modules.

ccetl avatar May 27 '24 01:05 ccetl

200 lines is on the edge of being too long, but it's better than having three files I think

superblaubeere27 avatar May 27 '24 01:05 superblaubeere27

~~I probably gonna split it in multiple files.~~

ccetl avatar Jun 01 '24 21:06 ccetl

image

ccetl avatar Jun 02 '24 09:06 ccetl

Breadcrumbs was mostly intended as a way to track movement (hence the name, from Hansel and Gretel). There should be an option to retain them until the module is disabled. Now, they are only a visual gimmick.

SenkJu avatar Jun 03 '24 01:06 SenkJu

It was no option before either to keep them until the module is disabled again. 🤔 There was the max length setting.

Beside that you are able to set it as line, disable fade and make it only disappear after 10 seconds.

ccetl avatar Jun 03 '24 05:06 ccetl

We could bump the time to live for the vertices though.

superblaubeere27 avatar Jun 03 '24 20:06 superblaubeere27