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

MidnightConfig replaced with owo-lib #61

Merged
merged 8 commits into from
Jan 26, 2025

Conversation

MonkaKokosowa
Copy link
Contributor

I rewrote Resource Conditions to support owo lib class generation, now when there's a variable called saddleRecipe in the Model class that has annotation it will call saddleRecipe() on the config instance.

Also, since owo-lib also has issues with text clipping how about we shorten the name of config values and provide longer explanation using `.tooltip

obraz

@Flatkat
Copy link
Member

Flatkat commented Jan 13, 2025

Freaking owo leaves even less space for the names smh 😭
I wasnt expecting it to be that bad, i dont even know how we could summarize some features so they are short enough and they still make sense (even if we can explain them in tooltips)

# Conflicts:
#	src/main/java/dev/symphony/harmony/config/HarmonyConfig.java
#	src/main/java/dev/symphony/harmony/item/ModifyItems.java
#	src/main/java/dev/symphony/harmony/mixin/food/glow_berries_apply_glowing/GlowBerriesApplyGlowing.java
#	src/main/java/dev/symphony/harmony/mixin/redstone/copper_bulb_delay/OneTickCopperBulbDelay.java
#	src/main/java/dev/symphony/harmony/mixin/transportation/riptide/TridentItemMixin.java
#	src/main/java/dev/symphony/harmony/mixin/transportation/teleport/ender_pearls_dont_dismount/EnderPearlsDontDismount.java
#	src/main/java/dev/symphony/harmony/mixin/transportation/trident/riptide_accceleration/RiptideAcceleration.java
#	src/main/java/dev/symphony/harmony/mixin/transportation/vehicle/exit_vehicle_on_damage/ExitVehicleOnDamage.java
#	src/main/resources/assets/harmony/lang/en_us.json
Between owo-lib branch and main after having javadoc credits merged
@Flatkat Flatkat merged commit a09a114 into Team-Symphony:main Jan 26, 2025
1 check passed
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.

3 participants