Skip to content

Comments

NeoUI - Color widgets and Begin/EndPopup#1706

Merged
nullsystem merged 3 commits intoNeotokyoRebuild:masterfrom
nullsystem:GH-1484_NeoUIColorWidget
Feb 21, 2026
Merged

NeoUI - Color widgets and Begin/EndPopup#1706
nullsystem merged 3 commits intoNeotokyoRebuild:masterfrom
nullsystem:GH-1484_NeoUIColorWidget

Conversation

@nullsystem
Copy link
Collaborator

Description

Color widgets shows the color from given RGBA values and acts as a button to show a popup where the RGBA values can be altered. This also changed how popups are handled, they are now similar to sections and can hold the same widgets and layouts as sections but have specialized handling for popup functionalities.

Keyboard and controller can exit popup with ESC/back-related bind, and showing popup on first initialization activates on the first item of the popup, although this is an optional behavior only utilized for color widgets.

Copy, server, and blacklist menus all migrated to using the new Begin/EndPopup.

Toolchain

  • Linux GCC Distro Native Arch/GCC 15

Linked Issues

@nullsystem nullsystem force-pushed the GH-1484_NeoUIColorWidget branch from 84a67e5 to 794a6f0 Compare February 20, 2026 00:37
Color widgets shows the color from given RGBA values and acts as
a button to show a popup where the RGBA values can be altered.
This also changed how popups are handled, they are now similar
to sections and can hold the same widgets and layouts as sections
but have specialized handling for popup functionalities.

Keyboard and controller can exit popup with ESC/back-related bind,
and showing popup on first initialization activates on the first
item of the popup, although this is an optional behavior only
utilized for color widgets.

Copy, server, and blacklist menus all migrated to using the new
Begin/EndPopup.

* fixes NeotokyoRebuild#1484
@nullsystem nullsystem force-pushed the GH-1484_NeoUIColorWidget branch from 794a6f0 to 005926c Compare February 20, 2026 01:02
Rainyan
Rainyan previously approved these changes Feb 21, 2026
Copy link
Collaborator

@Rainyan Rainyan left a comment

Choose a reason for hiding this comment

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

LGTM

As a nit, it would be nice to be able to move to next/previous field within a popup using tab and shift+tab, similar to how the up/down arrow keys work. For example, after typing a color Red value, it would be nice to be able to press tab to move the cursor to the Green value and so on.

@Rainyan Rainyan requested a review from a team February 21, 2026 02:22
Allow up/down to also be done with tab key, although restricted for
popups only. TODO in the future might actually change current
non-popup tab behavior from switching sections to also up/down and
up/down goes through all sections not just one, but this is out
of scope of this pull request.
@nullsystem
Copy link
Collaborator Author

nullsystem commented Feb 21, 2026

@Rainyan Added ability to use tab for up/down in popup now: d5bee30
I might expand tab up/down to all sections in the future (and no longer tab as section switch) but that'll be out of scope of this PR for now.

EDIT: Also removed unused color variables: b89ebb3

@nullsystem nullsystem requested a review from Rainyan February 21, 2026 12:19
@Rainyan Rainyan requested a review from a team February 21, 2026 15:12
Copy link
Contributor

@sunzenshen sunzenshen left a comment

Choose a reason for hiding this comment

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

LGTM:

Image Image Image

@nullsystem nullsystem merged commit cbcc3b0 into NeotokyoRebuild:master Feb 21, 2026
7 checks 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.

NeoUI - Color widget

3 participants