Skip to content

Color screen edit#78

Open
shvaich wants to merge 9 commits into
Alexdoru:masterfrom
shvaich:color_screen_edit
Open

Color screen edit#78
shvaich wants to merge 9 commits into
Alexdoru:masterfrom
shvaich:color_screen_edit

Conversation

@shvaich

@shvaich shvaich commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

made changes to have the new color selection screen.
also made changes to 'GuiUtil.drawVerticalLine' and 'GuiUtil.drawHorizontalLine' once I saw that they add 1px unnecessarily.
even tho it might seem logical, I think it is a mistake and should not be done.
if the changes to this methods are pushed, some other render code that uses such methods may need to be changed

@Alexdoru

Copy link
Copy Markdown
Owner

please add screeshots

@shvaich

shvaich commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

This is not yet the final version, some code will change.
This branch has conflicts with config_button_ui_edit branch, I should update it if and after config_button_ui_edit is merged.

The blueish border of a slider means that it is focused.

  • Clicking on a slider focuses it
  • If a slider is focused and you click on something else, that slider loses focus
  • Pressing TAB cycles focus
  • When a slider is focused, you can add/subtract 1 using ArrowLeft and ArrowRight

Undo Changes and Reset to Default buttons work the same as with RendererEditScreen.

The black background is: GuiScreen#drawDefaultBackground(). (cancelable with Patcher)

Color with no alpha (Opaque):

2026-07-20_22 43 03

Color with no alpha (Opaque) - With a focused slider:

2026-07-20_22 43 06

Color with alpha:

2026-07-20_22 43 20

Color with alpha - With a focused slider:

2026-07-20_22 43 26

I don't know if it is noticable in the "with alpha" examples I sent but the background of the Color Box in the middle of the screen is clear, it is not effected by the screen's background

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