Skip to content

[CG] Fix cg_fovViewModel when cg_fovAspectAdjust#1345

Open
SomaZ wants to merge 1 commit into
JACoders:masterfrom
SomaZ:fix_cg_fovviewmodel
Open

[CG] Fix cg_fovViewModel when cg_fovAspectAdjust#1345
SomaZ wants to merge 1 commit into
JACoders:masterfrom
SomaZ:fix_cg_fovviewmodel

Conversation

@SomaZ

@SomaZ SomaZ commented Jul 13, 2026

Copy link
Copy Markdown
Member

The scaling factor should always be the same and based on the difference of the set fov and the wanted fov. All fov effects can work properly now and arent overwritten and it works with aspect correction.

The scaling factor should always be the same and based on the difference of the set fov and the wanted fov. All fov effects can work properly now and arent overwritten and it works with aspect correction.
@SomaZ
SomaZ requested a review from a team as a code owner July 13, 2026 00:19
@ensiform

Copy link
Copy Markdown
Member

Doesn't this break cases where cg_fov is overridden

@SomaZ

SomaZ commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

Doesn't this break cases where cg_fov is overridden

This is exactly what it actually fixes. Currently, cases with overwritten fov will still force the exact defined weapon fov. This is not desirable though as you want the effect to apply to the weapon as well.

@SomaZ

SomaZ commented Jul 14, 2026

Copy link
Copy Markdown
Member Author

Running cg_fov 100 and cg_fovViewModel 0, using force speed:
shot2026-07-15_00-14-50
shot2026-07-15_00-14-39

running cg_fov 100 and cg_fovViewModel 101 with fixed code:
shot2026-07-15_00-15-10
shot2026-07-15_00-15-16

running cg_fov 100 and cg_fovViewModel 101 with broken code:
shot2026-07-15_00-26-17
shot2026-07-15_00-26-30

now broken code with disabled cg_fovViewModelAdjust to get the model at least in sight:
shot2026-07-15_00-26-48
shot2026-07-15_00-27-02
shot2026-07-15_00-33-01
shot2026-07-15_00-26-30

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