Skip to content

Blend the animated Wallpaper character into the CRT scene - #58

Open
xijiangkiki wants to merge 1 commit into
Code-Amadeus:mainfrom
xijiangkiki:ui/fix-wallpaper-character-color-balance
Open

Blend the animated Wallpaper character into the CRT scene#58
xijiangkiki wants to merge 1 commit into
Code-Amadeus:mainfrom
xijiangkiki:ui/fix-wallpaper-character-color-balance

Conversation

@xijiangkiki

Copy link
Copy Markdown
Contributor

What and why

Wallpaper mode combines a static CRT scene with a separately rendered animated foreground character.

The static in-scene character already fits naturally into the room. In contrast, the animated foreground character in Figure 2 is noticeably brighter and more saturated than the muted CRT scene, so it feels like a separate layer placed over the wallpaper and breaks the sense of immersion.

This PR applies a Wallpaper-only adjustment to the foreground character:

  • 95% brightness;
  • 10% lower saturation;
  • a slightly lower sprite position, without cropping the visible artwork.

The character remains the focal point, but the animated layer now fits the scene more naturally. Normal Chat rendering and character assets are unchanged.

Change class

  • Routine fix, documentation, test, maintenance, or presentation-only UI
  • Product-semantic or public-contract change discussed in the linked Issue
  • Isolated, default-off experiment

Owning layer: Wallpaper foreground rendering.

User-visible effect: Animated Wallpaper characters are slightly dimmer, less saturated, and positioned lower in the CRT viewport.

Compatibility or migration impact: none.

Before / after

Figure 1 — Static in-scene reference

The static character already shares the scene’s muted CRT treatment.

固定1

Figure 2 — Before

The animated foreground character is overly vivid against the aged CRT scene, appears to float above the screen rather than occupy the room, and feels visually detached from the environment.

动态

Figure 3 — After

The adjusted foreground character keeps Kurisu visually prominent while reducing the color separation and improving her placement within the CRT screen. Within the limits of the current assets and layout, it balances her role as the focal subject with a more natural integration into the background.

动态改

Evidence

Commands and manual journeys run:

  • node --check render/web/renderer.js
  • node --check render/web/wallpaper_scene.js
  • git diff --check
  • Electron npm run build
  • Before/after screenshots are attached for visible UI changes

Final check

  • This PR addresses one coherent presentation problem without unrelated cleanup
  • It does not add a speculative API, fallback, or compatibility path
  • No secrets, local state, model weights, voice material, or restricted assets are included
  • Third-party notices and provenance are preserved

@Lucas1479

Copy link
Copy Markdown
Member

Thanks for the thoughtful improvement — the color and brightness adjustments look reasonable and help the character blend into the CRT scene much better.

My only concern is the lower character position. The current screenshots show the character without the dialogue subtitle frame. When the frame is visible, it occupies the lower part of the CRT and is rendered above the character, so moving the character further down may cause more of her to be obscured.

Could you please add an after screenshot with a real two-line subtitle displayed? If the overlap still looks balanced, I’d be happy with this change. Thanks again for the contribution!

感谢这次细致的改进——颜色和亮度调整看起来很合理,人物与 CRT 场景的融合感确实好了很多。

我唯一有些担心的是人物位置下移。目前的截图展示的是没有对话字幕框的情况;字幕框显示时会占据 CRT 下方区域,并且位于人物图层之上,因此人物继续下移后,可能会有更多部分被遮挡。

可以麻烦补一张显示真实双行字幕时的调整后截图吗?如果这种情况下整体遮挡仍然协调,我觉得这个改动就没有问题。再次感谢你的贡献!

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