Camera view config options

[elexis]

Since it can be really useful for players and people who create promotional videos to change the maxzoom or camera movement speed, we should provide a way to change these things in the config page.

[view] ; Camera control settings
scroll.speed = 120.0
scroll.speed.modifier = 1.05      ; Multiplier for changing scroll speed
rotate.x.speed = 1.2
rotate.x.min = 28.0
rotate.x.max = 60.0
rotate.x.default = 35.0
rotate.y.speed = 2.0
rotate.y.speed.wheel = 0.45
rotate.y.default = 0.0
rotate.speed.modifier = 1.05      ; Multiplier for changing rotation speed
drag.speed = 0.5
zoom.speed = 256.0
zoom.speed.wheel = 32.0
zoom.min = 50.0
zoom.max = 200.0
zoom.default = 120.0
zoom.speed.modifier = 1.05        ; Multiplier for changing zoom speed
pos.smoothness = 0.1
zoom.smoothness = 0.4
rotate.x.smoothness = 0.5
rotate.y.smoothness = 0.3
near = 2.0                        ; Near plane distance
far = 4096.0                      ; Far plane distance
fov = 45.0                        ; Field of view (degrees), lower is narrow, higher is wide
height.smoothness = 0.5
height.min = 16

Assuming that the options are only visible if some "advanced settings" boolean is toggled and assuming that we have tabbing (as used in the credits page for example), these options could be exposed in the options page with checkboxes and sliders (potentially in a separate patch).

Independently, the Restrict camera option in developer_overlay.xml should also be loaded from a config entry, so that artists wanting to take screenshots from arbitrary angles can do that without having to interact with the menu each start (potentially in a separate patch).

[Stan]

Refs #3653 (Dupe ?)

[elexis]

Ah thanks. Iso-view part is dupe. The camera options are a requirement to implement it in the least messy way afaics.

[elexis]

The isometric view part should be reserved for #3653.

In the context of this ticket, people can code the config options and potentially the options page entries.

[elexis]

The biggest task will be to find a good subset of the options, the ones that are useful to the player (to me most importantly keyboard scroll speed and farthest zoom level).

If we can avoid the verbosity of the config file, we maybe don't need the "advanced settings" checkbox.

[Anil Eapen]

Screenshot of #4638 in progress

[elexis]

The isometric view should really not be done using config options, but the other settings look good.

[Stan]

Replying to elexis:

The isometric view should really not be done using config options.

Why not ?

[elexis]

See the comments in #3653, it should be done with an different OpenGL view projection.

[Stan]

Could still be an option with greater impact right ?

[Anil Eapen]

Replying to elexis:

See the comments in #3653 , it should be done with an different OpenGL view projection.

Implementation of isometric view will be taken care of as per #3653

but how is the player going to enable Isometric view if not in the config options ?

[elexis]

Sure, if there is new C++ code, there will have to be a new boolean in the config.

[Stan]

Sorry I missed you today on IRC.

To submit your patch you have to follow the guidelines. The patch has to be submitted through Phabricator. Code.wildfiregames.com

To have your name in contributors your patch needs to be reviewed there accepted and committed

[Anil Eapen]

Replying to stanislas69:

Sorry I missed you today on IRC.

To submit your patch you have to follow the guidelines. The patch has to be submitted through Phabricator. Code.wildfiregames.com

To have your name in contributors your patch needs to be reviewed there accepted and committed

As per ​https://trac.wildfiregames.com/wiki/Phabricator

'For security reasons, we require patch authors to be members of the Contributors group on Phabricator before their patches are automatically tested. So before uploading your first patch, just come on IRC and say hello so a team member can add you to the group.'

Anyways i'll submit the patch on Phabricator then.

[Stan]

My bad then :) You have been added anyways :)

[elexis]

"'For security reasons, we require patch authors to be members of the Contributors group on Phabricator before their patches are automatically tested. So before uploading your first patch, just come on IRC and say hello so a team member can add you to the group.'"

Password Hi, I saw it.

[Anil Eapen]

Please find patch uploaded on ​Phab:D1493 for review.

[Anil Eapen]

Replying to stanislas69:

My bad then :) You have been added anyways :)

Thanksyou :)

[Stan]

Backlogging due to lack of progress.

[Langbart]

#6150 was closed as a duplicate