Skip to content

Added tradex feature to clear highlighting manually and automatically#6

Open
CreepilyCreeper wants to merge 8 commits intoGjum:mainfrom
CreepilyCreeper:main
Open

Added tradex feature to clear highlighting manually and automatically#6
CreepilyCreeper wants to merge 8 commits intoGjum:mainfrom
CreepilyCreeper:main

Conversation

@CreepilyCreeper
Copy link
Contributor

No description provided.

Copy link
Owner

@Gjum Gjum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, great additions, needs cleanup before merge

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add to gitignore

fabric.mod.json Outdated
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this file supposed to be here?

Comment on lines 26 to 31
/**
* Positions that have been permanently purged (by distance or time).
* They will not be re-shown until the user triggers highlight creation again
* (e.g. clicking a chest or pressing "Highlight search results").
*/
private final HashSet<Pos> purgedPositions = new HashSet<>();
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll gonna redo the implementation maybe

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

Successfully merging this pull request may close these issues.

2 participants

Comments