filipf added a comment.
Thanks for the new version, almost there.
INLINE COMMENTS
> KeyboardButton.qml:25
> + style: QQCS.MenuStyle {
> + frame: Rectangle {
> + color: PlasmaCore.ColorScope.backgroundColor
One more thing that I realized is missing here and that would make everything
look like the desktop menus is a 1px border.
Now the stolen code should sort of look like this:
property color borderColor: PlasmaCore.ColorScope.textColor
border.color: Qt.rgba(borderColor.r, borderColor.g, borderColor.b, 0.3)
border.width: 1
Unfortunately for some reason rgba fails when using dark themes and the border
just turns white. Maybe you could figure out something better, but if you get
stuck it's not super important.
> KeyboardButton.qml:39
> + visible: styleData.selected
> + color: "#3daee9"
> + }
We still need to remove this bit of hardcoding:
`color: PlasmaCore.ColorScope.highlightColor`
and in the other file as well, of course.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D21794
To: cblack, #vdg, filipf
Cc: apol, plasma-devel, LeGast00n, jraleigh, GB_2, ragreen, Pitel, ZrenBot,
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart