skybox: improve target color selection UX
This commit is contained in:
@@ -92,6 +92,7 @@ public class SkyboxPlugin extends Plugin
|
|||||||
config.setColorToOverride(c);
|
config.setColorToOverride(c);
|
||||||
}
|
}
|
||||||
config.setOverrideMode(SkyOverrideMode.ONE);
|
config.setOverrideMode(SkyOverrideMode.ONE);
|
||||||
|
config.setPickColorToOverride(false);
|
||||||
}
|
}
|
||||||
|
|
||||||
skybox.setOverrideColors(config.colorToOverride().getRGB(), config.customColor().getRGB());
|
skybox.setOverrideColors(config.colorToOverride().getRGB(), config.customColor().getRGB());
|
||||||
|
|||||||
@@ -38,7 +38,10 @@ public interface SkyboxPluginConfig extends Config
|
|||||||
description = "Set a color here to use it with the options below",
|
description = "Set a color here to use it with the options below",
|
||||||
position = 1
|
position = 1
|
||||||
)
|
)
|
||||||
default Color customColor() { return Color.BLACK; }
|
default Color customColor()
|
||||||
|
{
|
||||||
|
return Color.BLACK;
|
||||||
|
}
|
||||||
|
|
||||||
@ConfigItem(
|
@ConfigItem(
|
||||||
keyName = "skyOverrideMode",
|
keyName = "skyOverrideMode",
|
||||||
@@ -50,31 +53,40 @@ public interface SkyboxPluginConfig extends Config
|
|||||||
{
|
{
|
||||||
return SkyOverrideMode.NONE;
|
return SkyOverrideMode.NONE;
|
||||||
}
|
}
|
||||||
|
@ConfigItem(
|
||||||
|
keyName = "skyOverrideMode", name = "", description = ""
|
||||||
|
)
|
||||||
void setOverrideMode(SkyOverrideMode value);
|
void setOverrideMode(SkyOverrideMode value);
|
||||||
|
|
||||||
@ConfigItem(
|
@ConfigItem(
|
||||||
keyName = "pickColorToOverride",
|
keyName = "pickColorToOverride",
|
||||||
name = "Set current sky as target",
|
name = "Set current sky as target",
|
||||||
description = "Sets target color to current chunk's sky color when switched on",
|
description = "Sets target color to current chunk's sky color when switched on",
|
||||||
|
warning = "Are you sure that you want to set the target color to the current sky color and forget the previous one?",
|
||||||
position = 3
|
position = 3
|
||||||
)
|
)
|
||||||
default boolean pickColorToOverride() { return false; }
|
default boolean pickColorToOverride()
|
||||||
|
{
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
@ConfigItem(
|
||||||
|
keyName = "pickColorToOverride", name = "", description = ""
|
||||||
|
)
|
||||||
|
void setPickColorToOverride(boolean value);
|
||||||
|
|
||||||
@ConfigItem(
|
@ConfigItem(
|
||||||
keyName = "colorToOverride",
|
keyName = "colorToOverride",
|
||||||
name = "Target color",
|
name = "Target color",
|
||||||
description = "Changing this manually is not recommended, but copying the value for tweaking is useful. Note that the value will not update when you enable the picker, until you re-enter the config panel.",
|
description = "Changing this manually is not recommended, but copying the value for tweaking is useful. Note that the value will not update when you enable the picker, until you re-enter the config panel.",
|
||||||
hidden = false,
|
hidden = true,
|
||||||
position = 4
|
position = 4
|
||||||
)
|
)
|
||||||
default Color colorToOverride() { return Color.BLACK; }
|
default Color colorToOverride()
|
||||||
|
{
|
||||||
|
return Color.BLACK;
|
||||||
|
}
|
||||||
@ConfigItem(
|
@ConfigItem(
|
||||||
keyName = "colorToOverride",
|
keyName = "colorToOverride", name = "", description = ""
|
||||||
name = "Target color",
|
|
||||||
description = "Changing this manually is not recommended, but copying the value for tweaking is useful. Note that the value will not update when you enable the picker, until you re-enter the config panel.",
|
|
||||||
hidden = false,
|
|
||||||
position = 4
|
|
||||||
)
|
)
|
||||||
void setColorToOverride(Color value);
|
void setColorToOverride(Color value);
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user