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

Feature: Highlight More Hoppity's Collection Rabbits #2438

Merged
merged 15 commits into from
Sep 15, 2024

Conversation

The1Divider
Copy link
Contributor

@The1Divider The1Divider commented Aug 31, 2024

What

Added a draggable list to highlight rabbits in Hoppity's collection with conditions,
Rewrote existing rabbit lore matching from onBackgroundDrawn, rabbits are cached in onInventoryOpen + added highlighting for shop/factory milestones, strays and abi.
Removed highlightRabbitsWithRequirement and onlyHighlightRequirementNotMet toggles

Colors for milestones/strays/abi are placeholders

Images

image
image

Changelog New Features

  • Added a draggable list for Hoppity's Collection menu rabbit highlighting. - the1divider
    • Factory/Shop milestones = Yellow/Gold.
    • Stray rabbits = Dark Aqua.
    • Abi = Dark Green.
  • Added a toggle to highlight found rabbits in Hoppity's Collection menu. - the1divider

Changelog Technical Details

  • Hoppity's Collection highlighting now updates on event triggers instead of every frame. - the1divider

Changelog Removed Features

  • Removed Only Requirement Not Met and Highlight Requirement Rabbits config options. - the1divider
    • Replaced then with a draggable list of different Hoppity collection highlight options.

@hannibal002 hannibal002 added this to the Version 0.27 milestone Sep 3, 2024
@jani270 jani270 added the Soon This Pull Request will be merged within the next couple of betas label Sep 9, 2024
@hannibal002 hannibal002 merged commit b8f4b63 into hannibal002:beta Sep 15, 2024
4 checks passed
@github-actions github-actions bot removed the Soon This Pull Request will be merged within the next couple of betas label Sep 15, 2024
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.

3 participants