Skip to content

feat: add canvas resolution configuration #412

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

HardyNLee
Copy link

@HardyNLee HardyNLee commented May 10, 2025

介绍

新增 画布分辨率 配置选项,预览窗口可垂直调整大小,

  • 英文翻译
  • 日文翻译

此 PR 以 OpenWebGAL/WebGAL#687 为前提

image
image

注意

  • 修改画布分辨率后,需要刷新游戏以生效
  • 预览窗口宽高比小于 1:1 时,画面会发生旋转,需自行调整预览窗口比例

@MakinoharaShoko
Copy link
Member

image
这个实际上有更加优雅的实现方式,用全局状态管理(Terre 使用了 zustand),记录游戏配置的宽高比,然后可以自动根据宽度算出来高度。如果你不清楚如何实现,请等待我们将其补充。

@MakinoharaShoko
Copy link
Member

另外,我提议用一个单独的配置项来设置是否启用分辨率调整。当这个配置项启用后,再展开调整分辨率的选项。并且,设置是否启用分辨率调整的配置项,使用类似于下图的方法,警告创作者功能的限制和可能引发的问题。

image

@HardyNLee
Copy link
Author

这个实际上有更加优雅的实现方式,用全局状态管理(Terre 使用了 zustand),记录游戏配置的宽高比,然后可以自动根据宽度算出来高度。

这里实际上是我有意为之,因为我认为不应该锁预览窗口宽高比,此修改是为了让用户更自由地调整预览窗口大小

另外,我提议用一个单独的配置项来设置是否启用分辨率调整。当这个配置项启用后,再展开调整分辨率的选项。并且,设置是否启用分辨率调整的配置项,使用类似于下图的方法,警告创作者功能的限制和可能引发的问题。

关于这点,其实我觉得加个开关有点太绕了,可否在旁边加一个恢复到默认值的按钮,外加醒目提示,这样更方便一点

@MakinoharaShoko
Copy link
Member

可否在旁边加一个恢复到默认值的按钮,外加醒目提示,这样更方便一点

这个我认为是可行的。

这里实际上是我有意为之,因为我认为不应该锁预览窗口宽高比,此修改是为了让用户更自由地调整预览窗口大小

这个不可行,因为这样会导致用户调整水平方向的栏时,每次都需要重新调整竖直方向的栏,如果用户频繁调整,非常麻烦。

@MakinoharaShoko
Copy link
Member

而且,我就是那种会频繁调整栏的宽度的那种人,因为我习惯写一段,拉大看看效果,然后拉小,继续写。

@HardyNLee
Copy link
Author

HardyNLee commented May 13, 2025

这个实际上有更加优雅的实现方式,用全局状态管理(Terre 使用了 zustand),记录游戏配置的宽高比,然后可以自动根据宽度算出来高度。

关于这点,其实我觉得加个开关有点太绕了,可否在旁边加一个恢复到默认值的按钮,外加醒目提示,这样更方便一点

已修改
image

@HardyNLee HardyNLee force-pushed the feat-canvas-resolution branch from af0e298 to cddb7d9 Compare May 13, 2025 14:45
update resolution setting UI

update translation

fix canvas resolution not set appropriately when game started

remove unnecessary import

remove unnecessary css property

update locales
@HardyNLee HardyNLee force-pushed the feat-canvas-resolution branch from cddb7d9 to 80e8287 Compare May 13, 2025 17:18
@HardyNLee HardyNLee marked this pull request as draft June 13, 2025 10:08
@HardyNLee HardyNLee marked this pull request as draft June 13, 2025 10:08
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