Skip to content

qt: Introduce platform specific css sections - #3570

Merged
UdjinM6 merged 3 commits into
dashpay:developfrom
xdustinface:pr-ui-17-platform-css
Jul 20, 2020
Merged

qt: Introduce platform specific css sections#3570
UdjinM6 merged 3 commits into
dashpay:developfrom
xdustinface:pr-ui-17-platform-css

Conversation

@xdustinface

@xdustinfacexdustinface commented Jun 26, 2020

Copy link
Copy Markdown

This PR ist part of a series of +-25 PRs related to UI redesigns. Its ancestor is #3569, its successor is #3571. I did not screenshot every single PR and its changes, instead i made "walk through all screen" videos with the result of this PR series and also with the 0.15 UI. If there are any concrete screenshots wanted, just let me know. To build with the full set of changes you can build from the branch xdustinface:pr-ui-redesign which always contains all changes.

-> Walk through 0.15
-> Walk through Redesign

I tried to give the commits enough text to make things obvious without a lot description for each PR. Also here, if you want more description for this specific PR, let me know.

About this PR

See commit message.

@xdustinface
xdustinfaceforce-pushed the pr-ui-17-platform-css branch 2 times, most recently from 3b86e75 to 936d38dCompareJune 27, 2020 20:23
@xdustinface
xdustinface marked this pull request as draft June 30, 2020 20:21
@xdustinface
xdustinfaceforce-pushed the pr-ui-17-platform-css branch from 936d38d to 372166aCompareJuly 17, 2020 11:00
This commit leads to GUIUtil::loadStyleSheet treating css code between
<os="<os_list>"> and </os> different. It will only become added for
operating systems provided in the list of the sections start tag.
There may be multiple entries per section. Possible entries:
- macosx
- windows
- other
<os_list> must be a combination of the three options above separated by
comma like in "windows,macosx".
Its ok to have multiple <os="...">...</os> sections in a file with
arbitrary OS combinations. They will all become added to the end of the
file though. Means even putting an <os> section in the top of the file
would become appended to the end of the file during loading which should
be kept in mind when adding sections to avoid unexpected overwriting.
Example
------------------------------------------------------------------------
<os="macosx, windows, other">
/* Example section to add styles for all operating systems
Remove any to exclude it.
*/
</os>
@xdustinface
xdustinfaceforce-pushed the pr-ui-17-platform-css branch from 372166a to 88f4d0bCompareJuly 17, 2020 11:00
@xdustinface
xdustinface marked this pull request as ready for review July 17, 2020 11:01
@xdustinface

Copy link
Copy Markdown
Author

Ready for review!

Comment threadsrc/qt/guiutil.cpp Outdated
Comment threadsrc/qt/guiutil.cpp Outdated
@UdjinM6UdjinM6 added this to the 17 milestone Jul 17, 2020
UdjinM6
UdjinM6 previously approved these changes Jul 17, 2020

@UdjinM6UdjinM6 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@PastaPastaPasta
PastaPastaPasta self-requested a review July 18, 2020 20:40
Comment threadsrc/qt/guiutil.cpp
PastaPastaPasta
PastaPastaPasta previously approved these changes Jul 19, 2020

@PastaPastaPastaPastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@UdjinM6UdjinM6 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

re-ACK

@PastaPastaPastaPastaPastaPasta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@UdjinM6
UdjinM6 merged commit 415e504 into dashpay:developJul 20, 2020
xdustinface added a commit to xdustinface/dash that referenced this pull request Sep 1, 2020
* qt: Add platform specific css loading to GUIUtil::loadStyleSheet
This commit leads to GUIUtil::loadStyleSheet treating css code between
<os="<os_list>"> and </os> different. It will only become added for
operating systems provided in the list of the sections start tag.
There may be multiple entries per section. Possible entries:
- macosx
- windows
- other
<os_list> must be a combination of the three options above separated by
comma like in "windows,macosx".
Its ok to have multiple <os="...">...</os> sections in a file with
arbitrary OS combinations. They will all become added to the end of the
file though. Means even putting an <os> section in the top of the file
would become appended to the end of the file during loading which should
be kept in mind when adding sections to avoid unexpected overwriting.
Example
------------------------------------------------------------------------
<os="macosx, windows, other">
/* Example section to add styles for all operating systems
Remove any to exclude it.
*/
</os>
* Respect `-uiplatform` when matching for os-specific styles
* Format osStyleExp to make it a bit easier to see groups
Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com>
@UdjinM6UdjinM6 modified the milestones: 17, 16Sep 4, 2020
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Mar 3, 2022
* qt: Add platform specific css loading to GUIUtil::loadStyleSheet
This commit leads to GUIUtil::loadStyleSheet treating css code between
<os="<os_list>"> and </os> different. It will only become added for
operating systems provided in the list of the sections start tag.
There may be multiple entries per section. Possible entries:
- macosx
- windows
- other
<os_list> must be a combination of the three options above separated by
comma like in "windows,macosx".
Its ok to have multiple <os="...">...</os> sections in a file with
arbitrary OS combinations. They will all become added to the end of the
file though. Means even putting an <os> section in the top of the file
would become appended to the end of the file during loading which should
be kept in mind when adding sections to avoid unexpected overwriting.
Example
------------------------------------------------------------------------
<os="macosx, windows, other">
/* Example section to add styles for all operating systems
Remove any to exclude it.
*/
</os>
* Respect `-uiplatform` when matching for os-specific styles
* Format osStyleExp to make it a bit easier to see groups
Co-authored-by: UdjinM6 <UdjinM6@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@xdustinface@UdjinM6@PastaPastaPasta