Closed RectangularObject closed 5 months ago
Reverted the cleanup since it's not really related to the pr, I'm assuming @Sychic or @My-Name-Is-Jeff will clean up the code in a future commit.
Hey, thanks for the PR. While this is a solution to the problem, a preferred solution would be to provide configuration for each gui element separately. This could be achieved by adding a field to the base GuiElement
class and reading from there. Toggling the option could then be handled by right clicking the element.
Toggling the option could then be handled by right clicking the element.
Huh, I didn't think of this. I'll work on it.
If you do, please try to use an elementa state to contain the value so it's easier to migrate later.
Well, there we go. It works, you can go into the configuration file and change every element's text shadow setting individually. Now I just have to figure out how to change that setting in-game using a right or middle click on the element. Is it a bad time to say that I've never really used Kotlin or Java before? Running off of pure intuition here. I'll have to look into how to actually use elementa tomorrow; I don't know much about it. It looks interesting!
Well, there we go. It works, you can go into the configuration file and change every element's text shadow setting individually. Now I just have to figure out how to change that setting in-game using a right or middle click on the element. Is it a bad time to say that I've never really used Kotlin or Java before? Running off of pure intuition here. I'll have to look into how to actually use elementa tomorrow; I don't know much about it. It looks interesting!
Cool, it works.
Is this still being looked at?
Is this still being looked at?
Forgot about it ngl will review rn
This PR has 177
quantified lines of changes. In general, a change size of upto 200
lines is ideal for the best PR experience!
Was this comment helpful? :thumbsup: :ok_hand: :thumbsdown: (Email) Customize PullRequestQuantifier for this repository.
Hooray for a pr on a suggestion made 2 years ago! This replaces the spam text shadow config with an option to change all hud text shadows, and will allow skytils' hud elements to match other mods, such as Apec. Also cleaned up the code a bit using intellij's "Reformat Code" function.
Ideally, I would've liked to implement config options to change each hud element's font shadow individually, but oh well! This'll do for now. Maybe I'll do that whenever I find the motivation to.