Skip to content

Fix demoui smoother buttons being unreactive#1842

Merged
Rainyan merged 3 commits intoNeotokyoRebuild:masterfrom
Rainyan:bug/demosmoother-btn
Mar 17, 2026
Merged

Fix demoui smoother buttons being unreactive#1842
Rainyan merged 3 commits intoNeotokyoRebuild:masterfrom
Rainyan:bug/demosmoother-btn

Conversation

@Rainyan
Copy link
Collaborator

@Rainyan Rainyan commented Mar 15, 2026

Description

Fix the demoui demo smoother's dropdown buttons in the legacy VGUI not responding to mouse clicks, making the smoother unusable.

If this gets refactored in the future, then

#ifdef NEO // NEO NOTE (Adam) mzync changed how button activation type is assigned in 1a6b546e, restore button activation type here
m_pOptions->SetButtonActivationType(Button::ACTIVATE_ONPRESSED);
#endif // NEO
should probably also be revisited.

Toolchain

  • Windows MSVC VS2022

Linked Issues

@Rainyan Rainyan requested a review from a team March 15, 2026 10:07
@sunzenshen sunzenshen requested a review from a team March 15, 2026 17:01
sunzenshen
sunzenshen previously approved these changes Mar 15, 2026
@sunzenshen sunzenshen requested a review from a team March 15, 2026 18:56
@DESTROYGIRL DESTROYGIRL added the UI/HUD Relates to the HUD, NeoUI, menus, etc label Mar 15, 2026
@Rainyan Rainyan force-pushed the bug/demosmoother-btn branch from 1ef0d7c to d73d34b Compare March 15, 2026 23:51
Improve code readability
@Rainyan Rainyan force-pushed the bug/demosmoother-btn branch from afca445 to 31678e2 Compare March 15, 2026 23:52
@Rainyan Rainyan requested a review from sunzenshen March 16, 2026 00:27
@sunzenshen sunzenshen requested a review from a team March 16, 2026 01:05
@AdamTadeusz
Copy link
Contributor

on an adjacent topic I wonder if the drive button is broken too. I seem to remember it working like described here where the button is pressed and held, and the player can move around and rotate the camera while the button is held, but currently in nt;re you have to press it to toggle it, release the left mouse button and then press and hold the button again, and only then you can move the camera around.

@Rainyan
Copy link
Collaborator Author

Rainyan commented Mar 17, 2026

on an adjacent topic I wonder if the drive button is broken too. I seem to remember it working like described here where the button is pressed and held, and the player can move around and rotate the camera while the button is held, but currently in nt;re you have to press it to toggle it, release the left mouse button and then press and hold the button again, and only then you can move the camera around.

Hmm, I'm too unfamiliar with how the tool is supposed to work but it's possible. Should probably by filed as its own bug with repro steps.

@Rainyan Rainyan merged commit a7f815c into NeotokyoRebuild:master Mar 17, 2026
7 checks passed
@Rainyan Rainyan deleted the bug/demosmoother-btn branch March 17, 2026 09:13
AdamTadeusz pushed a commit to AdamTadeusz/neo that referenced this pull request Mar 19, 2026
* Fix demoui smoother buttons being unreactive

* Fix wrong budget group

* Refactor

Improve code readability
AdamTadeusz added a commit that referenced this pull request Mar 19, 2026
* Fix demoui smoother buttons being unreactive (#1842)

* Fix demoui smoother buttons being unreactive

* Fix wrong budget group

* Refactor

Improve code readability

* Server side second auto record command (#1855)

* working

* fix

* delay game_end to end of round

* not every comp game should necessarily be recorded

* rainfix

* Fix spec_next/prev not skipping dead players (#1856)

Fix a case where in server demos, spec_next/spec_prev does not correctly skip dead players.

* DebugDrawLine grenade trails for spectators and practicing (#1852)

* working using debugdralines

* clamp and float and stuff

* Fix env_sun being hidden when under the centre of view (#1839)

* Changing cl_neo_squad_hud_avatar_size updates star texture sizes immediately, also cl_neo_hud_centre convars (#1829)

* init

* also solve the woldpos marker centre sizes not updating automatically

* le epic linux fail

* maybe inline breaking it

* center -> centre, GetFloat, f

* HLTV Changes for a smoother demo watching experience (#1845)

* init

* cl_observercrosshair is only for roaming observer crosshair

* makes most of the new spectate commands work in hltv

* fix beacons too

* view offsets, userid to entindex properly

* neobotmanager ignore sourcetv, fix rest spectate commands

* crosshair assert when spectating dead players

* float, call spectateChecks function

* gpGlobals->maxClients

* nits plus quick fixes plus viewmodel is really dark in stvdemos which i only now noticed

* don't move lighting origin of weapons attached to players

* should probably define this string somewhere else at this point

* revert changes to hud_crosshair trying to be smart about cl_observercrosshair

* Update src/game/shared/neo/neo_gamerules.cpp

Co-authored-by: DESTROYGIRL <toddlodyte44@outlook.com>

---------

Co-authored-by: DESTROYGIRL <toddlodyte44@outlook.com>

---------

Co-authored-by: Rainyan <Rainyan@users.noreply.github.com>
Co-authored-by: DESTROYGIRL <toddlodyte44@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI/HUD Relates to the HUD, NeoUI, menus, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Demo UI smoother should work

4 participants