Opened 3 years ago

Last modified 13 months ago

#4638 new enhancement

Camera view config options

Reported by: elexis Owned by:
Priority: Should Have Milestone: Backlog
Component: UI – Miscellaneous Keywords:
Cc: Patch: Phab:D1493

Description (last modified by 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).

Attachments (1)

4638.png (1.1 MB ) - added by Anil Eapen 2 years ago.
Screenshot of #4638 in progress

Download all attachments as: .zip

Change History (25)

comment:1 by stanislas69, 3 years ago

Refs #3653 (Dupe ?)

comment:2 by elexis, 3 years ago

Milestone: Backlog
Resolution: duplicate
Status: newclosed

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

comment:3 by elexis, 3 years ago

Component: UI & SimulationCore engine
Description: modified (diff)
Keywords: simple added
Milestone: Backlog
Resolution: duplicate
Status: closedreopened
Summary: Camera Options and Isometric ViewCamera view config options

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.

comment:4 by elexis, 3 years ago

Description: modified (diff)

comment:5 by Anil Eapen, 3 years ago

Owner: set to Anil Eapen
Status: reopenednew

comment:6 by elexis, 3 years ago

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.

by Anil Eapen, 2 years ago

Attachment: 4638.png added

Screenshot of #4638 in progress

comment:7 by Anil Eapen, 2 years ago

Last edited 2 years ago by Anil Eapen (previous) (diff)

comment:8 by elexis, 2 years ago

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

in reply to:  8 comment:9 by stanislas69, 2 years ago

Replying to elexis:

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

Why not ?

comment:10 by elexis, 2 years ago

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

comment:11 by stanislas69, 2 years ago

Could still be an option with greater impact right ?

in reply to:  10 comment:12 by Anil Eapen, 2 years ago

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 ?

Last edited 2 years ago by Anil Eapen (previous) (diff)

comment:13 by elexis, 2 years ago

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

comment:14 by stanislas69, 2 years ago

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

in reply to:  14 comment:15 by Anil Eapen, 2 years ago

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.

comment:16 by stanislas69, 2 years ago

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

comment:17 by elexis, 2 years ago

"'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.

comment:18 by Anil Eapen, 2 years ago

Please find patch uploaded on Phab:D1493 for review.

in reply to:  16 comment:19 by Anil Eapen, 2 years ago

Replying to stanislas69:

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

Thanksyou :)

comment:20 by elexis, 2 years ago

Keywords: simple removed
Milestone: BacklogWork In Progress

comment:21 by Anil Eapen, 2 years ago

Status: newassigned

comment:22 by elexis, 16 months ago

Component: Core engineUI – Miscellaneous
Milestone: Work In ProgressAlpha 24
Priority: Nice to HaveShould Have
Type: defectenhancement

comment:23 by elexis, 16 months ago

Patch: Phab:D1493

comment:24 by stanislas69, 13 months ago

Milestone: Alpha 24Backlog
Owner: Anil Eapen removed
Status: assignednew

Backlogging due to lack of progress.

Note: See TracTickets for help on using tickets.