Skip to content

fix(chat): improve scrollbar and IME badge contrast - #6

Merged
hzqst merged 1 commit into
mainfrom
fix/issue-175-chat-colors
Oct 9, 2026
Merged

hzqst merged 1 commit into
mainfrom
fix/issue-175-chat-colors

Conversation

@hzqst

@hzqst hzqst commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Give the chat scrollbar a visible gray thumb, a dark track, and distinct normal/hover/pressed button backgrounds. Style the CHS language badge with white text on dark gray, avoiding the white-on-white appearance reported alongside the scrollbar issue.

Depends on MetaHookSv/MetaHook#11: the shared controls must consume these scheme keys for the palette to take effect. Merge that PR before this one for standalone builds, which fetch MetaHook main by default.

Related: MetaHookSv/MetaHookSv#175

Validation with the paired MetaHook controls:

  • Clean Debug and Release builds succeeded; 21-snapshot gamedata validation passed.
  • Format-check passed (30 files), and git diff --check passed.
  • Sven Co-op 10257 / stadium4 / 800x600: checked arrows, thumb dragging, hover and pressed states, and the CHS badge. Screenshot pixels confirmed button/track/thumb RGB 64/32/160, hover/pressed RGB 96/32, and badge background RGB 64.

RenderDoc capture was unavailable (unsupported glTexEnvf at startup; late injection caused exit 100); verification used halflife-mcp screenshots without injection. Shutdown returned 0xC0000409 and remains undiagnosed. Full IME composition, other games, and other plugins were not tested. Builds reported C4101 warnings in unchanged exportfuncs.cpp and Debug LNK4075.

Use contrasting button, track and thumb colors, including hover and pressed states, and a dark language badge with white text.

Co-Authored-By: Codex <codex@openai.com>
@hzqst
hzqst merged commit 08de79d into main Oct 9, 2026
3 checks passed
@hzqst
hzqst deleted the fix/issue-175-chat-colors branch October 9, 2026 09:09
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.

1 participant