-
Notifications
You must be signed in to change notification settings - Fork 8
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
Make all layout values configurable #9
base: master
Are you sure you want to change the base?
Conversation
Looks interesting. I'll test it when I have time. Also a documentation should be written. I'll list the tasks for myself:
|
Documenting the new config options now |
@RadRussianRus here is a full doc for all the new options, not sure how to add this to the wiki Area Widths
Offset
Spacing
Skew
Image Padding
Slices Padding
User List Item Padding
CODE:
|
Fixing now, I had no way of testing that, I have discovered the debug options now though |
@RadRussianRus it's fixed but it's not pixel-perfect to the original. It looks fine though, just slight different positioning due to less random |
Looks good enough. |
* "Layout Scheme" renamed to "Size Scheme" to match internal name * Fixed typos * Added links to PR and wiki with quick description
I've adapted README to match its other sections. |
FYI: Next step is to make a generic |
@RadRussianRus can we push this forward? Really want to keep adding features without having to worry about the merge |
I understand, but currently I'm busy. Don't worry, I'll return to this PR when I have time. |
padding_vertical_slices=3 | ||
padding_vertical_slices_login_buttons=2 | ||
offset_vertical_slices_login_buttons=2 | ||
offset_horizontal_slices_login_buttons=0 | ||
|
||
padding_horizontal_item_session=10 | ||
padding_vertical_item_session=5 | ||
|
||
padding_horizontal_item_power=10 | ||
padding_vertical_item_power=5 | ||
|
||
padding_top_item_user=0 | ||
padding_bottom_item_user=-4 | ||
padding_horizontal_item_user=10 | ||
image_padding_item_user=4 | ||
offset_sub_head_item_user=-9 | ||
offset_lower_case_item_user=7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
theme.conf
is only for required settings, however in SizeScheme.qml
these settings are on higher levels, which means that they should fall back to Layer 1 values. Defining these values in theme.conf
prevents falling back to Layer 1.
There are multiple ways to solve this.
- These values can be moved to Layer 1 in
SizeScheme.qml
. However, global values may be affected not in a good way, so it needs to be checked. - Some new options may be introduced, like additional paddings or multipliers.
- These values also can be removed from
theme.conf
and hardcoded. It may be not a good fit for layout values scheme, but it should work as a temporary solution at least.
Sorry for long delay, got overloaded too much, but I still intend to get this PR merged. |
Related to #7
This PR makes all layout values, such as the offset, spacing, and padding configurable via the config. Most of the arbitrary values have been removed, and instead the size and shape of the interface elements are calculated from config values.
This has been tested and should be a pixel perfect to the original, however the power page hasn't been tested.