8000 Configurable default window dimensions by felipeerias · Pull Request #1828 · Igalia/wolvic · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Configurable default window dimensions #1828

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: main
Choose a base branch
from

Conversation

felipeerias
Copy link
Collaborator

Define the preset window dimensions at build time.

Unfortunately, Gradle doesn't allow us to define complex data structures and there isn't a clean way to access dimension resources from static constants and enums, which means that each dimension is defined as a separate constant.

The default size is WINDOW_PRESET_0_WIDTH by WINDOW_PRESET_0_HEIGHT.

We also define the DEFAULT_LOGICAL_WIDTH in CSS pixels. This value, along witht he density and the width of the default window are used to calculate the DPI.

This provides a configurable and predictable way of defining the default window size, logical size, and texture size, for different builds.

Define the preset window dimensions at build time.

Unfortunately, Gradle doesn't allow us to define complex data structures
and there isn't a clean way to access dimension resources from static
constants and enums, which means that each dimension is defined as a
separate constant.

The default size is WINDOW_PRESET_0_WIDTH by WINDOW_PRESET_0_HEIGHT.

We also define the DEFAULT_LOGICAL_WIDTH in CSS pixels. This value,
along witht he density and the width of the default window are used
to calculate the DPI.

This provides a configurable and predictable way of defining the default
window size, logical size, and texture size, for different builds.
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.

1 participant
0