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.

Sign up for GitHub

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

Jump to bottom

Show repair cost without advanced tooltips #81

Closed
sisby-folk opened this issue May 18, 2022 · 1 comment · Fixed by #94
Closed

Show repair cost without advanced tooltips #81

sisby-folk opened this issue May 18, 2022 · 1 comment · Fixed by #94

Comments

@sisby-folk
Copy link
Contributor

I'd like to show players in my pack the prior work penalty, to help them understand the cost of anvil repairs, but I don't want to force them to see item IDs to do it.

Would it be possible to add another option to show repair multiplier regardless of F3+H?
@LambdAurora
Copy link
Collaborator

Fixed in version 1.5.0 of the mod.

This was referenced Jul 29, 2022
Merged
Merged
LambdAurora pushed a commit that referenced this issue Jul 29, 2022
@sisby-folk
Remove Advanced Tooltips Check (#94)
55ab362 
Fixes #81. in-mixin check was removed, but not the vanilla one. 
This also makes lodestones show up without advanced (we're assuming this was intended)
LambdAurora pushed a commit that referenced this issue Jul 29, 2022
@sisby-folk
Fix #81 on 1.18.2 (#93)
2d9c4b0 
* Fix #81 on 1.18.2

Fixes #81 for the 1.18.2 branch.

No obligation to release this one, it'll just be what we use in our modpack, so I figured i'd PR it in case anyone else wants at it.

* Change onGetTooltip injection to Return
NebelNidas pushed a commit to ReviversMC/advanced-tooltips that referenced this issue Jul 29, 2022
@sisby-folk @NebelNidas
Remove Advanced Tooltips Check (Queerbric#94)
0a1b68b 
Fixes Queerbric#81. in-mixin check was removed, but not the vanilla one. 
This also makes lodestones show up without advanced (we're assuming this was intended)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove Advanced Tooltips Check (Fixes #81)
2 participants
@LambdAurora @sisby-folk