-
-
Notifications
You must be signed in to change notification settings - Fork 223
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: Optimal Angles Overlay for Squeaky Mousemat #2999
base: beta
Are you sure you want to change the base?
Conversation
#2977, might want to merge these two together in some way |
This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes. |
…gles # Conflicts: # src/main/java/at/hannibal2/skyhanni/utils/LorenzUtils.kt
Conflicts have been resolved! 🎉 |
@CalMWolfs can this be merged now? |
Ive marked this PR as soon, it should get merged in beta 1.3.0 or 1.4.0 or so |
I ment because it sais unresolver conversations, is there a way to make this go away if I created the pull request? |
The request was to remove a function from LorenzUtils as this class shouldn't be modified. I've already relocated the affected methods in a new class |
One or more Detekt Failures were detected:
|
I need this utils changes! |
This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes. |
…gles # Conflicts: # src/main/java/at/hannibal2/skyhanni/data/GuiEditManager.kt # src/main/java/at/hannibal2/skyhanni/features/garden/GardenOptimalSpeed.kt # src/main/java/at/hannibal2/skyhanni/utils/LorenzUtils.kt
Conflicts have been resolved! 🎉 |
This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes. |
…gles # Conflicts: # src/main/java/at/hannibal2/skyhanni/features/misc/BetterSignEditing.kt
Conflicts have been resolved! 🎉 |
Any status on this pr? |
I plan on looking at it tomorrow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the code works fine!
the variable names in the config and the display names are a bit confusing, maybe look over them again while creating the new mousemat config file.
the display here is a direct copy of the rancher boots gui. This gui uses a old design. please add hover text and different colors, maybe even a table (for having all values cleanly aligned) here. look at sack display render for an idea. if you need help with any of that feel free to ask
import io.github.notenoughupdates.moulconfig.annotations.ConfigLink | ||
import io.github.notenoughupdates.moulconfig.annotations.ConfigOption | ||
|
||
class OptimalAnglesConfig { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please move the squeaky mousemat options into a new class that is part of this class here
|
||
val signText = (this as AccessorGuiEditSign).tileSign.signText.map { it.unformattedText.removeColor() } | ||
return signText[1] == "^^^^^^" && signText[2] == "Set your" && signText[3] == "speed cap!" | ||
@Deprecated("do not use List<Any>, use List<Renderable> instead", ReplaceWith("")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here some merge conflict got wrongfully fixed. remove this function
|
||
private val config get() = GardenApi.config.optimalAngles | ||
|
||
private val configCustomAngles get() = config.customAngles |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if this is only used once it can be inlined
|
||
private fun CropType.getConfig(): Pair<Property<Float>, Property<Float>> = with(configCustomAngles) { | ||
when (this@getConfig) { | ||
CropType.CACTUS -> Pair(cactusYaw, cactusPitch) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of Pair(a, b)
you can use a to b
var signPosition: Position = Position(20, -195, false, true) | ||
|
||
@Expose | ||
@ConfigOption(name = "Custom Speed", desc = "Change the exact speed for every single crop.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update the name/desc
What
Adds an overlay for Squeaky Mousemat to choose from given angle-pairs for each crop - like the Rancher Boots Overlay for speed. You can edit the angles in config and choose between normal and compact display
Changelog New Features