mirror of
https://github.com/chylex/Better-Controls.git
synced 2026-02-15 06:06:12 +01:00
Compare commits
15 Commits
5111ac5f7e
...
main
| Author | SHA1 | Date | |
|---|---|---|---|
|
a8cc4e115b
|
|||
|
13fc85ff1c
|
|||
|
1c7c5402ad
|
|||
|
cdb9dadaec
|
|||
|
76afa6bc40
|
|||
|
cbf5adb4a7
|
|||
|
5a67786d0b
|
|||
|
2e19fffe44
|
|||
|
32f6a6e7fe
|
|||
|
dfefab3482
|
|||
|
ffd56a4407
|
|||
|
6b574f8568
|
|||
|
76a1353dbf
|
|||
|
0757d30f12
|
|||
|
64c4b51f0e
|
@@ -15,7 +15,7 @@ import javax.annotation.Nullable;
|
||||
@Mod("bettercontrols")
|
||||
public final class BetterControlsMod {
|
||||
public BetterControlsMod() {
|
||||
if (FMLEnvironment.dist == Dist.CLIENT) {
|
||||
if (FMLEnvironment.getDist() == Dist.CLIENT) {
|
||||
BetterControlsCommon.setConfig(BetterControlsConfig.load(FMLPaths.CONFIGDIR.get().resolve("BetterControls.json")));
|
||||
ModLoadingContext.get().registerExtensionPoint(IConfigScreenFactory.class, () -> BetterControlsMod::createOptionsScreen);
|
||||
}
|
||||
|
||||
@@ -33,7 +33,7 @@ You can also bind a key that resets all **Toggle Keybinds** at once. That makes
|
||||
#### Flying
|
||||
|
||||
* **Double Tap 'Jump' To Fly** can be turned off.
|
||||
* **Disable Flight Inertia** stops you instantly when you stop holding movement keys.
|
||||
* **Flight Inertia Multiplier** changes how quickly you stop moving in the air when you stop holding movement keys.
|
||||
* **Disable Field Of View Changing** prevents sprinting, potions, and other factors from changing the FOV while flying in creative and spectator mode.
|
||||
* **Fly On Ground** lets you fly while touching the ground in creative mode. Stop flying by tapping Sneak while touching the ground.
|
||||
* **Flight Speed Multiplier** (0.25x - 8x) changes flight speed in creative and spectator mode.
|
||||
|
||||
@@ -28,7 +28,7 @@ You can also bind a key that resets all **Toggle Keybinds** at once. That makes
|
||||
#### Flying
|
||||
|
||||
* **Double Tap 'Jump' To Fly** can be turned off.
|
||||
* **Disable Flight Inertia** stops you instantly when you stop holding movement keys.
|
||||
* **Flight Inertia Multiplier** changes how quickly you stop moving in the air when you stop holding movement keys.
|
||||
* **Disable Field Of View Changing** prevents sprinting, potions, and other factors from changing the FOV while flying in creative and spectator mode.
|
||||
* **Fly On Ground** lets you fly while touching the ground in creative mode. Stop flying by tapping Sneak while touching the ground.
|
||||
* **Flight Speed Multiplier** (0.25x - 8x) changes flight speed in creative and spectator mode.
|
||||
@@ -28,7 +28,7 @@ You can also bind a key that resets all **Toggle Keybinds** at once. That makes
|
||||
#### Flying
|
||||
|
||||
* **Double Tap 'Jump' To Fly** can be turned off.
|
||||
* **Disable Flight Inertia** stops you instantly when you stop holding movement keys.
|
||||
* **Flight Inertia Multiplier** changes how quickly you stop moving in the air when you stop holding movement keys.
|
||||
* **Disable Field Of View Changing** prevents sprinting, potions, and other factors from changing the FOV while flying in creative and spectator mode.
|
||||
* **Fly On Ground** lets you fly while touching the ground in creative mode. Stop flying by tapping Sneak while touching the ground.
|
||||
* **Flight Speed Multiplier** (0.25x - 8x) changes flight speed in creative and spectator mode.
|
||||
|
||||
@@ -3,18 +3,18 @@ modId=bettercontrols
|
||||
modName=Better Controls
|
||||
modDescription=Adds many powerful key bindings and options to control your movement.\\n\\nThe features complement vanilla mechanics without giving unfair advantages, so server use should be fine.
|
||||
modAuthor=chylex
|
||||
modVersion=1.5.0
|
||||
modVersion=1.6.4
|
||||
modLicense=MPL-2.0
|
||||
modSourcesURL=https://github.com/chylex/Better-Controls
|
||||
modIssuesURL=https://github.com/chylex/Better-Controls/issues
|
||||
modSides=client
|
||||
|
||||
# Dependencies
|
||||
minecraftVersion=1.21.4
|
||||
neoForgeVersion=21.4.77-beta
|
||||
neoModDevVersion=2.0.76
|
||||
fabricVersion=0.16.10
|
||||
loomVersion=1.9
|
||||
minecraftVersion=1.21.11
|
||||
neoForgeVersion=21.11.12-beta
|
||||
neoModDevVersion=2.0.110
|
||||
fabricVersion=0.17.2
|
||||
loomVersion=1.10
|
||||
mixinVersion=0.12.5+mixin.0.8.5
|
||||
mixinExtrasVersion=0.4.1
|
||||
|
||||
@@ -24,8 +24,8 @@ mixinExtrasVersion=0.4.1
|
||||
# https://github.com/FabricMC/fabric-loom/releases
|
||||
|
||||
# Constraints
|
||||
minimumMinecraftVersion=1.21.4
|
||||
minimumNeoForgeVersion=21.4.0-beta
|
||||
minimumMinecraftVersion=1.21.11
|
||||
minimumNeoForgeVersion=21.11.0-beta
|
||||
minimumFabricVersion=0.15.0
|
||||
|
||||
# Gradle
|
||||
|
||||
@@ -32,7 +32,7 @@ public final class BetterControlsConfig {
|
||||
|
||||
public final KeyBindingWithModifier keyToggleFlight = new KeyBindingWithModifier("key.bettercontrols.toggle_flight");
|
||||
public boolean doubleTapJumpToToggleFlight = true;
|
||||
public boolean disableFlightInertia = false;
|
||||
public float flightInertiaMultiplier = 1F;
|
||||
public boolean disableChangingFovWhileFlying = false;
|
||||
public boolean flyOnGroundInCreative = false;
|
||||
public float flightHorizontalSpeedMpCreativeDefault = 1F;
|
||||
|
||||
@@ -46,7 +46,7 @@ final class ConfigSerializer implements JsonSerializer<BetterControlsConfig>, Js
|
||||
|
||||
Json.writeKeyBinding(obj, "Flight.KeyToggle.Creative", cfg.keyToggleFlight);
|
||||
Json.setBool(obj, "Flight.DoubleTapJump", cfg.doubleTapJumpToToggleFlight);
|
||||
Json.setBool(obj, "Flight.DisableInertia", cfg.disableFlightInertia);
|
||||
Json.setFloat(obj, "Flight.InertiaMultiplier", cfg.flightInertiaMultiplier);
|
||||
Json.setBool(obj, "Flight.DisableChangingFOV", cfg.disableChangingFovWhileFlying);
|
||||
Json.setBool(obj, "Flight.FlyOnGround.Creative", cfg.flyOnGroundInCreative);
|
||||
Json.setFloat(obj, "Flight.SpeedMp.Creative.Default", cfg.flightHorizontalSpeedMpCreativeDefault);
|
||||
@@ -75,6 +75,10 @@ final class ConfigSerializer implements JsonSerializer<BetterControlsConfig>, Js
|
||||
cfg.sprintMode = SprintMode.TAP_TO_TOGGLE;
|
||||
}
|
||||
|
||||
if (obj.has("Flight.DisableInertia") && obj.get("Flight.DisableInertia").getAsBoolean()) {
|
||||
cfg.flightInertiaMultiplier = 0F;
|
||||
}
|
||||
|
||||
Json.readKeyBinding(obj, "Sprint.KeyToggle", cfg.keyToggleSprint);
|
||||
cfg.sprintMode = Json.getEnum(obj, "Sprint.Mode", cfg.sprintMode, SprintMode.class);
|
||||
cfg.doubleTapForwardToSprint = Json.getBool(obj, "Sprint.DoubleTapForward", cfg.doubleTapForwardToSprint);
|
||||
@@ -88,7 +92,7 @@ final class ConfigSerializer implements JsonSerializer<BetterControlsConfig>, Js
|
||||
|
||||
Json.readKeyBinding(obj, "Flight.KeyToggle.Creative", cfg.keyToggleFlight);
|
||||
cfg.doubleTapJumpToToggleFlight = Json.getBool(obj, "Flight.DoubleTapJump", cfg.doubleTapJumpToToggleFlight);
|
||||
cfg.disableFlightInertia = Json.getBool(obj, "Flight.DisableInertia", cfg.disableFlightInertia);
|
||||
cfg.flightInertiaMultiplier = Json.getFloat(obj, "Flight.InertiaMultiplier", cfg.flightInertiaMultiplier, 0F, 1F);
|
||||
cfg.disableChangingFovWhileFlying = Json.getBool(obj, "Flight.DisableChangingFOV", cfg.disableChangingFovWhileFlying);
|
||||
cfg.flyOnGroundInCreative = Json.getBool(obj, "Flight.FlyOnGround.Creative", cfg.flyOnGroundInCreative);
|
||||
cfg.flightHorizontalSpeedMpCreativeDefault = readHorizontalSpeedMultiplier(obj, "Flight.SpeedMp.Creative.Default", cfg.flightHorizontalSpeedMpCreativeDefault);
|
||||
|
||||
@@ -18,7 +18,10 @@ import net.minecraft.client.gui.components.CycleButton;
|
||||
import net.minecraft.client.gui.components.events.GuiEventListener;
|
||||
import net.minecraft.client.gui.screens.Screen;
|
||||
import net.minecraft.client.gui.screens.options.OptionsSubScreen;
|
||||
import net.minecraft.client.input.KeyEvent;
|
||||
import net.minecraft.client.input.MouseButtonEvent;
|
||||
import net.minecraft.network.chat.Component;
|
||||
import org.jetbrains.annotations.NotNull;
|
||||
import org.jetbrains.annotations.Nullable;
|
||||
import org.lwjgl.glfw.GLFW;
|
||||
import java.util.ArrayList;
|
||||
@@ -95,9 +98,18 @@ public class BetterControlsScreen extends OptionsSubScreen {
|
||||
return y;
|
||||
}
|
||||
|
||||
@SuppressWarnings({ "AutoBoxing", "AutoUnboxing" })
|
||||
private int generateFlightOptions(int y, List<GuiEventListener> elements) {
|
||||
BetterControlsConfig cfg = BetterControlsCommon.getConfig();
|
||||
|
||||
ImmutableList<Option<Float>> flightInertiaOptions = ImmutableList.of(
|
||||
new Option<>(Float.valueOf(0.00F), text("0x")),
|
||||
new Option<>(Float.valueOf(0.25F), text("0.25x")),
|
||||
new Option<>(Float.valueOf(0.50F), text("0.5x")),
|
||||
new Option<>(Float.valueOf(0.75F), text("0.75x")),
|
||||
new Option<>(Float.valueOf(1.00F), text("1x"))
|
||||
);
|
||||
|
||||
ImmutableList<Option<Float>> flightSpeedOptions = ImmutableList.of(
|
||||
new Option<>(Float.valueOf(0.25F), text("0.25x")),
|
||||
new Option<>(Float.valueOf(0.50F), text("0.5x")),
|
||||
@@ -119,7 +131,8 @@ public class BetterControlsScreen extends OptionsSubScreen {
|
||||
generateBooleanOptionRow(y, elements, text("Double Tap 'Jump' To Fly (Creative)"), cfg.doubleTapJumpToToggleFlight, value -> cfg.doubleTapJumpToToggleFlight = value);
|
||||
y += ROW_HEIGHT;
|
||||
|
||||
generateBooleanOptionRow(y, elements, text("Disable Flight Inertia"), cfg.disableFlightInertia, value -> cfg.disableFlightInertia = value);
|
||||
generateLeftSideText(y, elements, text("Flight Inertia Multiplier"));
|
||||
elements.add(new DiscreteValueSliderWidget<>(col2(1), y, COL2_W, text("Flight Inertia Multiplier"), flightInertiaOptions, cfg.flightInertiaMultiplier, value -> cfg.flightInertiaMultiplier = value));
|
||||
y += ROW_HEIGHT;
|
||||
|
||||
generateBooleanOptionRow(y, elements, text("Disable Field Of View Changing"), cfg.disableChangingFovWhileFlying, value -> cfg.disableChangingFovWhileFlying = value);
|
||||
@@ -201,9 +214,8 @@ public class BetterControlsScreen extends OptionsSubScreen {
|
||||
|
||||
private static void generateBooleanOptionRow(int y, List<GuiEventListener> elements, Component text, boolean initialValue, BooleanConsumer onValueChanged) {
|
||||
generateLeftSideText(y, elements, text);
|
||||
elements.add(CycleButton.onOffBuilder()
|
||||
elements.add(CycleButton.onOffBuilder(initialValue)
|
||||
.displayOnlyValue()
|
||||
.withInitialValue(Boolean.valueOf(initialValue))
|
||||
.create(col2(1), y, COL2_W, 20, text, (btn, newValue) -> onValueChanged.accept(newValue.booleanValue())));
|
||||
}
|
||||
|
||||
@@ -273,32 +285,32 @@ public class BetterControlsScreen extends OptionsSubScreen {
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean mouseClicked(double mouseX, double mouseY, int button) {
|
||||
public boolean mouseClicked(@NotNull MouseButtonEvent event, boolean isDoubleClick) {
|
||||
if (editingKeyBinding != null) {
|
||||
editingKeyBinding.bindAndStopEditing(InputConstants.Type.MOUSE.getOrCreate(button));
|
||||
editingKeyBinding.bindAndStopEditing(InputConstants.Type.MOUSE.getOrCreate(event.button()));
|
||||
onKeyBindingEditingFinished();
|
||||
return true;
|
||||
}
|
||||
else {
|
||||
return super.mouseClicked(mouseX, mouseY, button);
|
||||
return super.mouseClicked(event, isDoubleClick);
|
||||
}
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean keyPressed(int keyCode, int scanCode, int modifiers) {
|
||||
public boolean keyPressed(@NotNull KeyEvent event) {
|
||||
if (editingKeyBinding != null) {
|
||||
if (keyCode == GLFW.GLFW_KEY_ESCAPE) {
|
||||
if (event.key() == GLFW.GLFW_KEY_ESCAPE) {
|
||||
editingKeyBinding.bindAndStopEditing(InputConstants.UNKNOWN);
|
||||
}
|
||||
else {
|
||||
editingKeyBinding.bindAndStopEditing(InputConstants.getKey(keyCode, scanCode));
|
||||
editingKeyBinding.bindAndStopEditing(InputConstants.getKey(event));
|
||||
}
|
||||
|
||||
onKeyBindingEditingFinished();
|
||||
return true;
|
||||
}
|
||||
else {
|
||||
return super.keyPressed(keyCode, scanCode, modifiers);
|
||||
return super.keyPressed(event);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -9,12 +9,11 @@ import net.minecraft.client.gui.components.Renderable;
|
||||
import net.minecraft.client.gui.components.events.GuiEventListener;
|
||||
import net.minecraft.client.gui.narration.NarratableEntry;
|
||||
import org.jetbrains.annotations.NotNull;
|
||||
import java.util.ArrayList;
|
||||
import java.util.Collections;
|
||||
import org.jetbrains.annotations.Nullable;
|
||||
import java.util.List;
|
||||
import java.util.Map;
|
||||
import java.util.function.Function;
|
||||
import java.util.stream.Collectors;
|
||||
import static java.util.stream.Collectors.toMap;
|
||||
|
||||
public final class OptionListWidget extends ContainerObjectSelectionList<Entry> {
|
||||
public static final int ROW_WIDTH = 408;
|
||||
@@ -35,28 +34,32 @@ public final class OptionListWidget extends ContainerObjectSelectionList<Entry>
|
||||
|
||||
private static Offset getElementOffset(GuiEventListener element) {
|
||||
if (element instanceof OptionWidget widget) {
|
||||
return new Offset(widget.getX(), widget.getY());
|
||||
return new Offset(widget.getX(), widget.getY(), widget.getHeight());
|
||||
}
|
||||
else if (element instanceof AbstractWidget widget) {
|
||||
return new Offset(widget.getX(), widget.getY());
|
||||
return new Offset(widget.getX(), widget.getY(), widget.getHeight());
|
||||
}
|
||||
else {
|
||||
return new Offset(0, 0);
|
||||
return new Offset(0, 0, 0);
|
||||
}
|
||||
}
|
||||
|
||||
public interface OptionWidget extends GuiEventListener, Renderable {
|
||||
int getX();
|
||||
int getY();
|
||||
void setX(int x);
|
||||
int getX();
|
||||
|
||||
void setY(int y);
|
||||
int getY();
|
||||
|
||||
int getHeight();
|
||||
}
|
||||
|
||||
private record Offset(int x, int y) {}
|
||||
private record Offset(int x, int y, int height) {}
|
||||
|
||||
@SuppressWarnings("ThisEscapedInObjectConstruction")
|
||||
public OptionListWidget(int width, int height, int top, int innerHeight, List<GuiEventListener> widgets) {
|
||||
super(Minecraft.getInstance(), width, height, top, innerHeight);
|
||||
addEntry(new Entry(widgets));
|
||||
addEntry(new Entry(this, widgets));
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -80,41 +83,60 @@ public final class OptionListWidget extends ContainerObjectSelectionList<Entry>
|
||||
return true;
|
||||
}
|
||||
|
||||
@Override
|
||||
protected void scrollToEntry(@NotNull Entry entry) {
|
||||
// Scrolling to focused item is implemented in Entry.
|
||||
}
|
||||
|
||||
protected static final class Entry extends ContainerObjectSelectionList.Entry<Entry> {
|
||||
private final OptionListWidget parentWidget;
|
||||
private final List<GuiEventListener> elements;
|
||||
private final List<NarratableEntry> narratables;
|
||||
private final Map<GuiEventListener, Offset> offsets;
|
||||
|
||||
public Entry(List<GuiEventListener> elements) {
|
||||
this.elements = new ArrayList<>(elements);
|
||||
this.narratables = elements.stream().filter(e -> e instanceof NarratableEntry).map(e -> (NarratableEntry)e).collect(Collectors.toList());
|
||||
this.offsets = elements.stream().collect(Collectors.toMap(Function.identity(), OptionListWidget::getElementOffset));
|
||||
public Entry(OptionListWidget parentWidget, List<GuiEventListener> elements) {
|
||||
this.parentWidget = parentWidget;
|
||||
this.elements = List.copyOf(elements);
|
||||
this.narratables = elements.stream().filter(e -> e instanceof NarratableEntry).map(e -> (NarratableEntry)e).toList();
|
||||
this.offsets = elements.stream().collect(toMap(Function.identity(), OptionListWidget::getElementOffset));
|
||||
}
|
||||
|
||||
@Override
|
||||
public void setFocused(@Nullable GuiEventListener element) {
|
||||
super.setFocused(element);
|
||||
|
||||
if (Minecraft.getInstance().getLastInputType().isKeyboard()) {
|
||||
Offset offset = offsets.get(element);
|
||||
if (offset != null) {
|
||||
parentWidget.setScrollAmount(offset.y + (offset.height * 0.5F) - (parentWidget.getHeight() * 0.5F) + 4);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@NotNull
|
||||
@Override
|
||||
public List<? extends GuiEventListener> children() {
|
||||
return Collections.unmodifiableList(elements);
|
||||
return elements;
|
||||
}
|
||||
|
||||
@NotNull
|
||||
@Override
|
||||
public List<? extends NarratableEntry> narratables() {
|
||||
return Collections.unmodifiableList(narratables);
|
||||
return narratables;
|
||||
}
|
||||
|
||||
@Override
|
||||
public void render(@NotNull GuiGraphics graphics, int index, int y, int x, int entryWidth, int entryHeight, int mouseX, int mouseY, boolean hovered, float tickDelta) {
|
||||
public void renderContent(@NotNull GuiGraphics graphics, int mouseX, int mouseY, boolean hovered, float tickDelta) {
|
||||
for (GuiEventListener element : elements) {
|
||||
Offset offset = offsets.get(element);
|
||||
|
||||
if (element instanceof AbstractWidget widget) {
|
||||
widget.setX(x + offset.x);
|
||||
widget.setY(y + offset.y);
|
||||
widget.setX(getX() + offset.x);
|
||||
widget.setY(getY() + offset.y);
|
||||
}
|
||||
else if (element instanceof OptionWidget widget) {
|
||||
widget.setX(x + offset.x);
|
||||
widget.setY(y + offset.y);
|
||||
widget.setX(getX() + offset.x);
|
||||
widget.setY(getY() + offset.y);
|
||||
}
|
||||
|
||||
if (element instanceof Renderable renderable) {
|
||||
|
||||
@@ -2,9 +2,9 @@ package chylex.bettercontrols.gui.elements;
|
||||
|
||||
import com.google.common.collect.ImmutableList;
|
||||
import net.minecraft.client.gui.components.AbstractSliderButton;
|
||||
import net.minecraft.client.input.KeyEvent;
|
||||
import net.minecraft.network.chat.Component;
|
||||
import net.minecraft.network.chat.MutableComponent;
|
||||
import net.minecraft.util.Mth;
|
||||
import org.jetbrains.annotations.NotNull;
|
||||
import java.util.function.Consumer;
|
||||
|
||||
@@ -15,7 +15,7 @@ public final class DiscreteValueSliderWidget<T> extends AbstractSliderButton {
|
||||
private T selectedValue;
|
||||
|
||||
public DiscreteValueSliderWidget(int x, int y, int width, int height, Component narration, ImmutableList<Option<T>> options, T selectedValue, Consumer<T> onChanged) {
|
||||
super(x, y, width, height, Option.find(options, selectedValue).text(), options.indexOf(Option.find(options, selectedValue)) / (options.size() - 1.0));
|
||||
super(x, y, width, height, Option.find(options, selectedValue).text(), getOptionValue(options, options.indexOf(Option.find(options, selectedValue))));
|
||||
this.narration = narration;
|
||||
this.options = options;
|
||||
this.selectedValue = selectedValue;
|
||||
@@ -27,7 +27,11 @@ public final class DiscreteValueSliderWidget<T> extends AbstractSliderButton {
|
||||
}
|
||||
|
||||
public Option<T> getSelectedOption() {
|
||||
return options.get(Mth.floor(Mth.clampedLerp(0.0, options.size() - 1.0, value)));
|
||||
return options.get(getSelectedOptionIndex());
|
||||
}
|
||||
|
||||
private int getSelectedOptionIndex() {
|
||||
return getOptionIndex(value, options.size());
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -35,6 +39,29 @@ public final class DiscreteValueSliderWidget<T> extends AbstractSliderButton {
|
||||
setMessage(getSelectedOption().text());
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean keyPressed(KeyEvent keyEvent) {
|
||||
if (keyEvent.isSelection()) {
|
||||
return super.keyPressed(keyEvent);
|
||||
}
|
||||
|
||||
if (keyEvent.isLeft() || keyEvent.isRight()) {
|
||||
int newOptionIndex = keyEvent.isLeft()
|
||||
? getSelectedOptionIndex() - 1
|
||||
: getSelectedOptionIndex() + 1;
|
||||
|
||||
if (newOptionIndex >= 0 && newOptionIndex < options.size()) {
|
||||
value = getOptionValue(options, newOptionIndex);
|
||||
applyValue();
|
||||
updateMessage();
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
@Override
|
||||
protected void applyValue() {
|
||||
T newSelectedValue = getSelectedOption().value();
|
||||
@@ -50,4 +77,20 @@ public final class DiscreteValueSliderWidget<T> extends AbstractSliderButton {
|
||||
protected MutableComponent createNarrationMessage() {
|
||||
return Component.translatable("gui.narrate.slider", narration.plainCopy().append(" ").append(getMessage()));
|
||||
}
|
||||
|
||||
public static int getOptionIndex(double value, int optionCount) {
|
||||
if (value < 0.0) {
|
||||
return 0;
|
||||
}
|
||||
else if (value > 1.0) {
|
||||
return optionCount - 1;
|
||||
}
|
||||
else {
|
||||
return (int) (value * (optionCount - 1));
|
||||
}
|
||||
}
|
||||
|
||||
private static <T> double getOptionValue(ImmutableList<Option<T>> options, int optionIndex) {
|
||||
return optionIndex / (options.size() - 1.0);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -6,6 +6,8 @@ import net.minecraft.client.KeyMapping;
|
||||
import net.minecraft.client.Minecraft;
|
||||
import net.minecraft.client.gui.components.AbstractButton;
|
||||
import net.minecraft.client.gui.components.Button;
|
||||
import net.minecraft.client.gui.components.Tooltip;
|
||||
import net.minecraft.client.input.InputWithModifiers;
|
||||
import net.minecraft.network.chat.Component;
|
||||
import net.minecraft.network.chat.MutableComponent;
|
||||
import org.jetbrains.annotations.NotNull;
|
||||
@@ -13,7 +15,7 @@ import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
import java.util.function.Consumer;
|
||||
|
||||
public final class KeyBindingWidget extends Button {
|
||||
public final class KeyBindingWidget extends Button.Plain {
|
||||
private final KeyMapping binding;
|
||||
private final Component bindingName;
|
||||
|
||||
@@ -42,11 +44,13 @@ public final class KeyBindingWidget extends Button {
|
||||
@NotNull
|
||||
@Override
|
||||
protected MutableComponent createNarrationMessage() {
|
||||
return binding.isUnbound() ? Component.translatable("narrator.controls.unbound", bindingName) : Component.translatable("narrator.controls.bound", bindingName, super.createNarrationMessage());
|
||||
return binding.isUnbound()
|
||||
? Component.translatable("narrator.controls.unbound", bindingName)
|
||||
: Component.translatable("narrator.controls.bound", bindingName, super.createNarrationMessage());
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onPress() {
|
||||
public void onPress(@NotNull InputWithModifiers input) {
|
||||
isEditing = true;
|
||||
onEditingStarted.accept(this);
|
||||
updateKeyBindingText();
|
||||
@@ -68,24 +72,59 @@ public final class KeyBindingWidget extends Button {
|
||||
|
||||
public void updateKeyBindingText() {
|
||||
boolean hasConflict = false;
|
||||
MutableComponent conflictText = Component.empty();
|
||||
|
||||
if (!binding.isUnbound()) {
|
||||
for (KeyMapping other : Minecraft.getInstance().options.keyMappings) {
|
||||
if (binding != other && binding.same(other)) {
|
||||
if (hasConflict) {
|
||||
conflictText.append(", ");
|
||||
}
|
||||
|
||||
hasConflict = true;
|
||||
break;
|
||||
conflictText.append(Component.translatable(other.getName()));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (isEditing) {
|
||||
setMessage(Component.literal("> ").append(binding.getTranslatedKeyMessage().copy().withStyle(ChatFormatting.YELLOW)).append(" <").withStyle(ChatFormatting.YELLOW));
|
||||
}
|
||||
else if (hasConflict) {
|
||||
setMessage(binding.getTranslatedKeyMessage().copy().withStyle(ChatFormatting.RED));
|
||||
if (hasConflict) {
|
||||
setMessage(getMessageWithConflict(binding));
|
||||
setTooltip(Tooltip.create(Component.translatable("controls.keybinds.duplicateKeybinds", conflictText)));
|
||||
}
|
||||
else {
|
||||
setMessage(binding.isUnbound() ? Component.literal("(No Binding)") : binding.getTranslatedKeyMessage());
|
||||
setMessage(getMessageWithoutConflict(binding));
|
||||
setTooltip(null);
|
||||
}
|
||||
|
||||
if (isEditing) {
|
||||
setMessage(getEditingMessage(getMessage()));
|
||||
}
|
||||
}
|
||||
|
||||
private static MutableComponent getMessageWithConflict(KeyMapping binding) {
|
||||
return Component
|
||||
.literal("[ ")
|
||||
.append(binding.getTranslatedKeyMessage().copy().withStyle(ChatFormatting.WHITE))
|
||||
.append(" ]")
|
||||
.withStyle(ChatFormatting.YELLOW);
|
||||
}
|
||||
|
||||
private static Component getMessageWithoutConflict(KeyMapping binding) {
|
||||
if (binding.isUnbound()) {
|
||||
return Component
|
||||
.literal("(")
|
||||
.append(Component.translatable("key.keyboard.unknown"))
|
||||
.append(")");
|
||||
}
|
||||
|
||||
return binding.getTranslatedKeyMessage();
|
||||
}
|
||||
|
||||
private static MutableComponent getEditingMessage(Component originalMessage) {
|
||||
return Component
|
||||
.literal("> ")
|
||||
.append(originalMessage.copy().withStyle(ChatFormatting.WHITE, ChatFormatting.UNDERLINE))
|
||||
.append(" <")
|
||||
.withStyle(ChatFormatting.YELLOW);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -8,14 +8,13 @@ import java.util.function.Consumer;
|
||||
|
||||
public record Option<T>(T value, Component text) {
|
||||
public static <T> Option<T> find(List<Option<T>> options, T value) {
|
||||
return options.stream().filter(it -> Objects.equals(it.value, value)).findFirst().orElseGet(() -> options.get(0));
|
||||
return options.stream().filter(it -> Objects.equals(it.value, value)).findFirst().orElseGet(options::getFirst);
|
||||
}
|
||||
|
||||
public static <T> CycleButton<Option<T>> button(int x, int y, int width, Component text, List<Option<T>> options, T initialValue, Consumer<T> onValueChanged) {
|
||||
return CycleButton.<Option<T>>builder(Option::text)
|
||||
return CycleButton.builder(Option::text, find(options, initialValue))
|
||||
.displayOnlyValue()
|
||||
.withValues(options)
|
||||
.withInitialValue(find(options, initialValue))
|
||||
.create(x, y, width, 20, text, (btn, newValue) -> onValueChanged.accept(newValue.value()));
|
||||
}
|
||||
}
|
||||
|
||||
@@ -13,7 +13,7 @@ public final class TextWidget implements OptionWidget {
|
||||
public static final int LEFT = 0;
|
||||
public static final int CENTER = 1;
|
||||
|
||||
public static final int WHITE = 0xFF_FF_FF;
|
||||
public static final int WHITE = 0xFF_FF_FF_FF;
|
||||
|
||||
private final Component text;
|
||||
private int x;
|
||||
@@ -39,24 +39,29 @@ public final class TextWidget implements OptionWidget {
|
||||
this(x, y, width, 20, text, LEFT);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void setX(int x) {
|
||||
this.x = x;
|
||||
}
|
||||
|
||||
@Override
|
||||
public int getX() {
|
||||
return x;
|
||||
}
|
||||
|
||||
@Override
|
||||
public void setY(int y) {
|
||||
this.y = y;
|
||||
}
|
||||
|
||||
@Override
|
||||
public int getY() {
|
||||
return y;
|
||||
}
|
||||
|
||||
@Override
|
||||
public void setX(int x) {
|
||||
this.x = x;
|
||||
}
|
||||
|
||||
@Override
|
||||
public void setY(int y) {
|
||||
this.y = y;
|
||||
public int getHeight() {
|
||||
return height;
|
||||
}
|
||||
|
||||
@Override
|
||||
|
||||
@@ -2,13 +2,15 @@ package chylex.bettercontrols.input;
|
||||
|
||||
import com.mojang.blaze3d.platform.InputConstants.Type;
|
||||
import net.minecraft.client.KeyMapping;
|
||||
import net.minecraft.resources.Identifier;
|
||||
import org.jetbrains.annotations.Nullable;
|
||||
|
||||
public class KeyBindingWithModifier extends KeyMapping {
|
||||
public static final String CATEGORY = "key.categories.bettercontrols";
|
||||
@SuppressWarnings("SpellCheckingInspection")
|
||||
public static final Category CATEGORY = new Category(Identifier.fromNamespaceAndPath("bettercontrols", "all"));
|
||||
|
||||
@Nullable
|
||||
private ModifierKey modifier = null;
|
||||
private ModifierKey modifier;
|
||||
|
||||
public KeyBindingWithModifier(String translationKey) {
|
||||
super(translationKey, Type.KEYSYM, -1, CATEGORY);
|
||||
|
||||
@@ -1,26 +1,26 @@
|
||||
package chylex.bettercontrols.input;
|
||||
|
||||
import net.minecraft.client.gui.screens.Screen;
|
||||
import net.minecraft.client.Minecraft;
|
||||
|
||||
public enum ModifierKey {
|
||||
CONTROL(0) {
|
||||
@Override
|
||||
public boolean isPressed() {
|
||||
return Screen.hasControlDown();
|
||||
return Minecraft.getInstance().hasControlDown();
|
||||
}
|
||||
},
|
||||
|
||||
SHIFT(1) {
|
||||
@Override
|
||||
public boolean isPressed() {
|
||||
return Screen.hasShiftDown();
|
||||
return Minecraft.getInstance().hasShiftDown();
|
||||
}
|
||||
},
|
||||
|
||||
ALT(2) {
|
||||
@Override
|
||||
public boolean isPressed() {
|
||||
return Screen.hasAltDown();
|
||||
return Minecraft.getInstance().hasAltDown();
|
||||
}
|
||||
};
|
||||
|
||||
|
||||
@@ -3,16 +3,10 @@ package chylex.bettercontrols.mixin;
|
||||
import net.minecraft.client.KeyMapping;
|
||||
import org.spongepowered.asm.mixin.Mixin;
|
||||
import org.spongepowered.asm.mixin.gen.Accessor;
|
||||
import java.util.Map;
|
||||
|
||||
@Mixin(KeyMapping.class)
|
||||
@SuppressWarnings("StaticMethodOnlyUsedInOneClass")
|
||||
public interface AccessKeyMappingFields {
|
||||
@Accessor("CATEGORY_SORT_ORDER")
|
||||
static Map<String, Integer> getCategoryOrderMap() {
|
||||
throw new AssertionError();
|
||||
}
|
||||
|
||||
@Accessor("isDown")
|
||||
boolean isPressedField();
|
||||
|
||||
|
||||
@@ -2,7 +2,6 @@ package chylex.bettercontrols.mixin;
|
||||
|
||||
import chylex.bettercontrols.BetterControlsCommon;
|
||||
import chylex.bettercontrols.config.BetterControlsConfig;
|
||||
import chylex.bettercontrols.input.KeyBindingWithModifier;
|
||||
import net.minecraft.client.KeyMapping;
|
||||
import net.minecraft.client.Options;
|
||||
import org.apache.commons.lang3.ArrayUtils;
|
||||
@@ -38,6 +37,5 @@ public abstract class HookLoadGameOptions {
|
||||
|
||||
hasLoaded = true;
|
||||
keyMappings = ArrayUtils.addAll(keyMappings, config.getAllKeyBindings());
|
||||
AccessKeyMappingFields.getCategoryOrderMap().put(KeyBindingWithModifier.CATEGORY, Integer.valueOf(Integer.MAX_VALUE));
|
||||
}
|
||||
}
|
||||
|
||||
@@ -18,7 +18,7 @@ public abstract class HookStickyKeyBindingState extends KeyMapping {
|
||||
@Final
|
||||
private BooleanSupplier needsToggle;
|
||||
|
||||
public HookStickyKeyBindingState(String translationKey, int code, String category) {
|
||||
public HookStickyKeyBindingState(String translationKey, int code, KeyMapping.Category category) {
|
||||
super(translationKey, code, category);
|
||||
}
|
||||
|
||||
|
||||
@@ -87,7 +87,7 @@ public final class PlayerTicker {
|
||||
}
|
||||
|
||||
SprintMode sprintMode = getConfig().sprintMode;
|
||||
boolean wasSprintToggled = Boolean.TRUE.equals(OPTIONS.toggleSprint().get());
|
||||
boolean wasSprintToggled = OPTIONS.toggleSprint().get().booleanValue();
|
||||
boolean isSprintToggled = toggleSprint.tick();
|
||||
|
||||
if (temporarySprintTimer > 0) {
|
||||
@@ -223,15 +223,21 @@ public final class PlayerTicker {
|
||||
holdingSneakWhileTouchingGround = false;
|
||||
}
|
||||
|
||||
if (FlightHelper.isFlyingCreativeOrSpectator(player) && getConfig().disableFlightInertia) {
|
||||
ClientInput input = player.input;
|
||||
if (FlightHelper.isFlyingCreativeOrSpectator(player)) {
|
||||
float inertiaMultiplier = getConfig().flightInertiaMultiplier;
|
||||
|
||||
if (input.forwardImpulse == 0F && input.leftImpulse == 0F) {
|
||||
player.setDeltaMovement(player.getDeltaMovement().multiply(0.0, 1.0, 0.0));
|
||||
}
|
||||
|
||||
if (!input.keyPresses.jump() && !input.keyPresses.shift()) {
|
||||
player.setDeltaMovement(player.getDeltaMovement().multiply(1.0, 0.0, 1.0));
|
||||
if (inertiaMultiplier < 1F) {
|
||||
ClientInput input = player.input;
|
||||
Input keyPresses = input.keyPresses;
|
||||
double inertiaMultiplierSqrt = Math.sqrt(inertiaMultiplier);
|
||||
|
||||
if (!keyPresses.forward() && !keyPresses.backward() && !keyPresses.left() && !keyPresses.right()) {
|
||||
player.setDeltaMovement(player.getDeltaMovement().multiply(inertiaMultiplierSqrt, 1.0, inertiaMultiplierSqrt));
|
||||
}
|
||||
|
||||
if (!keyPresses.jump() && !keyPresses.shift()) {
|
||||
player.setDeltaMovement(player.getDeltaMovement().multiply(1.0, inertiaMultiplierSqrt, 1.0));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -264,7 +270,7 @@ public final class PlayerTicker {
|
||||
if (!getConfig().sneakingMovesCameraSmoothly) {
|
||||
Camera camera = MINECRAFT.gameRenderer.getMainCamera();
|
||||
|
||||
if (camera.getEntity() == player) {
|
||||
if (camera.entity() == player) {
|
||||
Mixins.cameraFields(camera).setEyeHeight(player.getEyeHeight());
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user