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

feat: scalable ch player arrow #521

Merged
merged 11 commits into from
Jan 27, 2025

Conversation

Kemmisch
Copy link
Contributor

make ch player arrow scalable through a config option

make ch player arrow scalable
Copy link
Member

@My-Name-Is-Jeff My-Name-Is-Jeff left a comment

Choose a reason for hiding this comment

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

also put it in the translation file

src/main/kotlin/gg/skytils/skytilsmod/core/Config.kt Outdated Show resolved Hide resolved
spelling & translation

Signed-off-by: songreaver  <[email protected]>
fix scaling numbers

Signed-off-by: songreaver  <[email protected]>
@Kemmisch
Copy link
Contributor Author

Kemmisch commented Nov 3, 2024

also put it in the translation file

should i leave CN & TW blank, or leave the english in there?

Signed-off-by: songreaver  <[email protected]>
move scale up and make dependent on map

Signed-off-by: songreaver  <[email protected]>
@My-Name-Is-Jeff
Copy link
Member

also put it in the translation file

should i leave CN & TW blank, or leave the english in there?

You can just leave it blank I believe and it should default to English

Kemmisch and others added 5 commits November 6, 2024 18:38
Co-authored-by: My-Name-Is-Jeff <[email protected]>
Signed-off-by: songreaver  <[email protected]>
Signed-off-by: songreaver  <[email protected]>
Signed-off-by: songreaver  <[email protected]>
oops missed an s

Signed-off-by: songreaver  <[email protected]>
ok apparently i forgot twice?

Signed-off-by: songreaver  <[email protected]>
@My-Name-Is-Jeff My-Name-Is-Jeff merged commit ecd98c7 into Skytils:dev Jan 27, 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.

2 participants