Changeset View
Standalone View
binaries/data/mods/public/gui/options/options.json
Show First 20 Lines • Show All 75 Lines • ▼ Show 20 Lines | [ | ||||
"min": 0, | "min": 0, | ||||
"max": 32 | "max": 32 | ||||
}, | }, | ||||
{ | { | ||||
"type": "boolean", | "type": "boolean", | ||||
"label": "Chat timestamp", | "label": "Chat timestamp", | ||||
"tooltip": "Show time that messages are posted in the lobby, gamesetup and ingame chat.", | "tooltip": "Show time that messages are posted in the lobby, gamesetup and ingame chat.", | ||||
"config": "chat.timestamp" | "config": "chat.timestamp" | ||||
}, | |||||
{ | |||||
"type": "dropdown", | |||||
"label": "GUI scale", | |||||
"tooltip": "GUI scale, REQUIRES GAME RESTART.", | |||||
"config": "gui.scale", | |||||
"list": [ | |||||
{ "value": "1", "label": "100%" }, | |||||
{ "value": "1.25", "label": "125%" }, | |||||
{ "value": "1.75", "label": "175%" }, | |||||
{ "value": "2", "label": "200%" } | |||||
] | |||||
} | } | ||||
] | ] | ||||
}, | }, | ||||
{ | { | ||||
"label": "Graphics", | "label": "Graphics", | ||||
"tooltip": "Set the balance between performance and visual appearance.", | "tooltip": "Set the balance between performance and visual appearance.", | ||||
"options": | "options": | ||||
[ | [ | ||||
{ | { | ||||
"type": "boolean", | "type": "boolean", | ||||
"label": "Windowed mode", | "label": "Windowed mode", | ||||
"tooltip": "Start 0 A.D. in a window.", | "tooltip": "Start 0 A.D. in a window.", | ||||
"config": "windowed" | "config": "windowed" | ||||
}, | }, | ||||
{ | { | ||||
"type": "boolean", | "type": "boolean", | ||||
"label": "Prefer GLSL", | "label": "Prefer GLSL", | ||||
"tooltip": "Use OpenGL 2.0 shaders (recommended).", | "tooltip": "Use OpenGL 2.0 shaders (recommended).", | ||||
"config": "preferglsl", | "config": "preferglsl", | ||||
vladislavbelov: Maybe leave a string type? To add a `Default` state or `System value` for a future. | |||||
Done Inline ActionsCant because c++ requires that to be number so it needs to be casted before sending to c++ Silier: Cant because c++ requires that to be number so it needs to be casted before sending to c++ | |||||
Not Done Inline ActionsYou might adjust the C++ part for that. vladislavbelov: You might adjust the C++ part for that. | |||||
"function": "Renderer_SetPreferGLSLEnabled" | "function": "Renderer_SetPreferGLSLEnabled" | ||||
}, | }, | ||||
{ | { | ||||
"type": "boolean", | "type": "boolean", | ||||
"label": "Fog", | "label": "Fog", | ||||
"tooltip": "Enable Fog.", | "tooltip": "Enable Fog.", | ||||
"dependencies": ["preferglsl"], | "dependencies": ["preferglsl"], | ||||
"config": "fog", | "config": "fog", | ||||
"function": "Renderer_SetFogEnabled" | "function": "Renderer_SetFogEnabled" | ||||
}, | }, | ||||
Not Done Inline Actions1.4 just fits into FullHD display given the current min resolution, 2.8 for 4k displays. Offering 1.5 seems a poor choice. sera: 1.4 just fits into FullHD display given the current min resolution, 2.8 for 4k displays. | |||||
Not Done Inline ActionsThere should be only divisors of 4. vladislavbelov: There should be only divisors of 4. | |||||
Not Done Inline ActionsSo make the multiplier 45/32 for FullHD and 45/16 for 4K sera: So make the multiplier 45/32 for FullHD and 45/16 for 4K | |||||
Not Done Inline Actions1920 / 45 or 3840 / 45 are not integers, so shouldn't be used to avoid too much rounding. vladislavbelov: 1920 / 45 or 3840 / 45 are not integers, so shouldn't be used to avoid too much rounding. | |||||
Not Done Inline ActionsTested and it doesn't look all that bad, scaling is a loss anyway. The main point is there should be an option labeled FullHD and one 4K, scaling up as much as possible for those resolutions. This to make picking easy and not break users exit button or similar due to over-sizing. Offering other choices is fine. 1.5 breaks the UI for FullHD, so unusable for most, which is clearly worse than a bit of rounding. Labeling 125% option FullHD works too, not sure people would prefer it tho. sera: Tested and it doesn't look all that bad, scaling is a loss anyway.
The main point is there… | |||||
Not Done Inline ActionsCould you post a screenshot, what's broken with 1.5? vladislavbelov: Could you post a screenshot, what's broken with 1.5? | |||||
Not Done Inline Actionssera: If you want 1.5 for FullHD you will have to set the min height for design to 720 (as well as… | |||||
Not Done Inline ActionsIn theory it's okay, because 1080 / 1.5 = 720 < 768 - our minimal support height. vladislavbelov: In theory it's okay, because 1080 / 1.5 = 720 < 768 - our minimal support height. | |||||
Not Done Inline ActionsUsers do not know/care about that now, do they ? Stan: Users do not know/care about that now, do they ? | |||||
Not Done Inline ActionsRedefining min height of 768 to 720 would also make sense for HDReady displays, if that is the route you'd like to take I have nothing to say against it. sera: Redefining min height of 768 to 720 would also make sense for HDReady displays, if that is the… | |||||
{ | { | ||||
"type": "boolean", | "type": "boolean", | ||||
"label": "Post-processing", | "label": "Post-processing", | ||||
"tooltip": "Use screen-space post-processing filters (HDR, Bloom, DOF, etc).", | "tooltip": "Use screen-space post-processing filters (HDR, Bloom, DOF, etc).", | ||||
"config": "postproc", | "config": "postproc", | ||||
"function": "Renderer_SetPostProcEnabled" | "function": "Renderer_SetPostProcEnabled" | ||||
}, | }, | ||||
{ | { | ||||
▲ Show 20 Lines • Show All 500 Lines • Show Last 20 Lines |
Maybe leave a string type? To add a Default state or System value for a future.