-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
use QColor for cue colors #2345
Changes from all commits
463135f
2de7846
be29b77
4e2da2d
5c31b98
3e9aca8
9498f16
b60f292
37541e9
37d5da1
aff6995
c756053
40825c4
b2c7c16
f72a679
1cadfea
d07b672
438b1c9
f6da0dd
606091c
9047343
ced08e2
cd3555e
525cd85
8e574d9
df545bf
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1566,7 +1566,6 @@ WPushButton:hover { | |
/*"Pressed" state*/ | ||
WPushButton[value="1"], | ||
WPushButton[value="2"] { | ||
/*color: #FDFDFD;*/ | ||
color: #FDFDFD; | ||
background-color: #006596; | ||
border: 0px solid #006596; | ||
|
@@ -1579,129 +1578,17 @@ WPushButton[value="2"]:hover { | |
border: 0px solid #0080BE; | ||
} | ||
|
||
/* Hotcue Color: No Color */ | ||
#HotcueButton[value="1"][highlight="0"], | ||
#HotcueButton[value="2"][highlight="0"] { | ||
background-color: #006596; | ||
} | ||
|
||
#HotcueButton[value="1"][highlight="0"]:hover, | ||
#HotcueButton[value="2"][highlight="0"]:hover { | ||
background-color: #0080BE; | ||
} | ||
|
||
/* Hotcue Color: Red */ | ||
#HotcueButton[value="1"][highlight="1"], | ||
#HotcueButton[value="2"][highlight="1"] { | ||
background-color: #c50a08; | ||
} | ||
|
||
#HotcueButton[value="1"][highlight="1"]:hover, | ||
#HotcueButton[value="2"][highlight="1"]:hover { | ||
background-color: #e50c08; | ||
} | ||
|
||
/* Hotcue Color: Green */ | ||
#HotcueButton[value="1"][highlight="2"], | ||
#HotcueButton[value="2"][highlight="2"] { | ||
background-color: #32be44; | ||
} | ||
|
||
#HotcueButton[value="1"][highlight="2"]:hover, | ||
#HotcueButton[value="2"][highlight="2"]:hover { | ||
background-color: #52de64; | ||
} | ||
|
||
/* Hotcue Color: Blue */ | ||
#HotcueButton[value="1"][highlight="3"], | ||
#HotcueButton[value="2"][highlight="3"] { | ||
background-color: #0044ff; | ||
} | ||
|
||
#HotcueButton[value="1"][highlight="3"]:hover, | ||
#HotcueButton[value="2"][highlight="3"]:hover { | ||
background-color: #0064ff; | ||
} | ||
|
||
/* Hotcue Color: Yellow */ | ||
#HotcueButton[value="1"][highlight="4"], | ||
#HotcueButton[value="2"][highlight="4"] { | ||
color: #4B4B4B; | ||
background-color: #f8d200; | ||
} | ||
|
||
#HotcueButton[value="1"][highlight="4"]:hover, | ||
#HotcueButton[value="2"][highlight="4"]:hover { | ||
color: #4B4B4B; | ||
background-color: #f8f200; | ||
} | ||
|
||
/* Hotcue Color: Celeste */ | ||
#HotcueButton[value="1"][highlight="5"], | ||
#HotcueButton[value="2"][highlight="5"] { | ||
color: #4B4B4B; | ||
background-color: #42d4f4; | ||
#HotcueButton { | ||
qproperty-shouldHighlightBackgroundOnHover: true; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This doesn't seem to be working. I see no color change when hovering hotcue buttons in Deere that have a hotcue set. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is weird, it works for me :/ There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 5.12.5 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This probably depends on the QStyle in use... Did you test with |
||
} | ||
|
||
#HotcueButton[value="1"][highlight="5"]:hover, | ||
#HotcueButton[value="2"][highlight="5"]:hover { | ||
color: #4B4B4B; | ||
background-color: #62f4f4; | ||
} | ||
|
||
/* Hotcue Color: Purple */ | ||
#HotcueButton[value="1"][highlight="6"], | ||
#HotcueButton[value="2"][highlight="6"] { | ||
background-color: #af00cc; | ||
} | ||
|
||
#HotcueButton[value="1"][highlight="6"]:hover, | ||
#HotcueButton[value="2"][highlight="6"]:hover { | ||
background-color: #cf00ec; | ||
} | ||
|
||
/* Hotcue Color: Pink */ | ||
#HotcueButton[value="1"][highlight="7"], | ||
#HotcueButton[value="2"][highlight="7"] { | ||
color: #4B4B4B; | ||
background-color: #fca6d7; | ||
} | ||
|
||
#HotcueButton[value="1"][highlight="7"]:hover, | ||
#HotcueButton[value="2"][highlight="7"]:hover { | ||
color: #4B4B4B; | ||
background-color: #fcc6f7; | ||
} | ||
|
||
/* Hotcue Color: White */ | ||
#HotcueButton[value="1"][highlight="8"], | ||
#HotcueButton[value="2"][highlight="8"] { | ||
color: #4B4B4B; | ||
background-color: #f2f2ff; | ||
} | ||
|
||
#HotcueButton[value="1"][highlight="8"]:hover, | ||
#HotcueButton[value="2"][highlight="8"]:hover { | ||
color: #4B4B4B; | ||
background-color: #ffffff; | ||
} | ||
|
||
/*"Enabled" state, e.g. for recording status | ||
0 -- disconnected / off | ||
1 -- connecting / enabling | ||
2 -- connected / enabled | ||
WPushButton[value="2"] { | ||
#HotcueButton[backgroundIsDark=true][hasBackgroundColor=true] { | ||
color: #FDFDFD; | ||
background-color: #4B4B4B; | ||
border: 0px solid #006596; | ||
} | ||
|
||
WPushButton[value="2"]:hover { | ||
color: #FDFDFD; | ||
background-color: #4B4B4B; | ||
border: 0px solid #0080BE; | ||
#HotcueButton[backgroundIsDark=false][hasBackgroundColor=true] { | ||
color: #1f1e1e; | ||
} | ||
*/ | ||
|
||
#PlayToggle[value="0"] { | ||
image: url(skin:/icon/ic_play_48px.svg) no-repeat center center; | ||
|
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.
I think it would be more convenient to use
color.colorFromHexCode
on the ControlObject value before passing it tothis.outputColor
.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.
Maybe @Holzhaus can help us here.
Can you have a look and check if I didn't break something?
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.
This does not work.
hotcue_X_color
contains an actual ARGB value, not the index of the color in the color palette.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.
The HotcueButtons stopped working as well. Let me check what causes this.
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.
Ok, the problem is that the
colors
property ofHotcueButton
now needs to be an object with the different colors as keys. Here's a diff: