Skip to content

Improve pattern provider grouping#8735

Open
Mithi83 wants to merge 1 commit intoAppliedEnergistics:mainfrom
Mithi83:fix-8732
Open

Improve pattern provider grouping#8735
Mithi83 wants to merge 1 commit intoAppliedEnergistics:mainfrom
Mithi83:fix-8732

Conversation

@Mithi83
Copy link
Copy Markdown
Collaborator

@Mithi83 Mithi83 commented Dec 4, 2025

If different group entries are available still use the more specific icon if it is the same for all group entries.

Fixes #8732

If different group entries are available still use the more specific
icon if it is the same for all group entries.

Fixes AppliedEnergistics#8732
@Mithi83
Copy link
Copy Markdown
Collaborator Author

Mithi83 commented Dec 4, 2025

Example with different combinations:

  1. Furnace + Molecular Assembler -> Generic Pattern Provider (as before), verbose tooltip (as before)
  2. 4 Molecular Assemblers with 5 Acceleration Cards each -> Molecular Assembler (as before), slim tooltip (as before)
  3. 4 Molecular Assemblers with no upgrades -> Molecular Assembler (as before), no tooltip (as before)
  4. 4 Molecular Assembler with 0, 2, 3, and 5 Acceleration Cards respectively -> Molecular Assembler (this has changed), verbose tooltip (as before)
image

For the review: I'm not sure if if (!icon.getId().equals(group.icon().getId())) is the best way to check for that equality, feel free to suggest an improved check.

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.

Molecular Assembler mislabeled in Pattern Access Terminal if it has uneven number of speed upgrades

1 participant