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: Optimal Angles Overlay for Squeaky Mousemat #2999

Open
wants to merge 26 commits into
base: beta
Choose a base branch
from

Conversation

nikijul007
Copy link

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

  • Optimal Angles Overlay. - nikijul

@hannibal002 hannibal002 added this to the Version 0.29 milestone Nov 28, 2024
@ILike2WatchMemes
Copy link
Contributor

#2977, might want to merge these two together in some way

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Dec 25, 2024
Copy link

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
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 1, 2025
Copy link

github-actions bot commented Jan 1, 2025

Conflicts have been resolved! 🎉

@rueblimaster
Copy link
Contributor

@CalMWolfs can this be merged now?

@hannibal002 hannibal002 added the Soon This Pull Request will be merged within the next couple of betas label Jan 7, 2025
@hannibal002
Copy link
Owner

Ive marked this PR as soon, it should get merged in beta 1.3.0 or 1.4.0 or so

@rueblimaster
Copy link
Contributor

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?

@nikijul007
Copy link
Author

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

@github-actions github-actions bot added the Detekt Has detekt problem label Jan 9, 2025
Copy link

github-actions bot commented Jan 9, 2025

One or more Detekt Failures were detected:

@github-actions github-actions bot removed the Detekt Has detekt problem label Jan 9, 2025
@rueblimaster
Copy link
Contributor

I need this utils changes!

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 25, 2025
Copy link

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
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot added Merge Conflicts There are open merge conflicts with the beta branch. and removed Merge Conflicts There are open merge conflicts with the beta branch. labels Jan 25, 2025
Copy link

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
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 25, 2025
@Eveeifyeve
Copy link

Any status on this pr?

@CalMWolfs
Copy link
Collaborator

Any status on this pr?

image

@NopoTheGamer
Copy link
Contributor

Any status on this pr?

image

@hannibal002
Copy link
Owner

I plan on looking at it tomorrow

Copy link
Owner

@hannibal002 hannibal002 left a 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
image

import io.github.notenoughupdates.moulconfig.annotations.ConfigLink
import io.github.notenoughupdates.moulconfig.annotations.ConfigOption

class OptimalAnglesConfig {
Copy link
Owner

@hannibal002 hannibal002 Feb 11, 2025

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(""))
Copy link
Owner

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
Copy link
Owner

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)
Copy link
Owner

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.")
Copy link
Owner

Choose a reason for hiding this comment

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

update the name/desc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Soon This Pull Request will be merged within the next couple of betas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants