Differential D19325 Diff 52864 lookandfeel/contents/components/Clock.qml

Changeset View

Standalone View

View Options

lookandfeel/contents/components/Clock.qml

Show All 31 Lines27Item {
32 width: clock.implicitWidth 32​ width: clock.implicitWidth
33 height: clock.implicitHeight 33​ height: clock.implicitHeight
34 34​
35 DropShadow { 35​ DropShadow {
36 id: clockShadow 36​ id: clockShadow
37 visible: !softwareRendering 37​ visible: !softwareRendering
38 anchors.fill: clock 38​ anchors.fill: clock
39 source: clock 39​ source: clock
40 horizontalOffset: 0 40​ horizontalOffset: 1
41 verticalOffset: 2 41​ verticalOffset: 1
42 radius: 14 42​ radius: 6
43 samples: 32 43​ samples: 14
44 spread: 0.3 44​ spread: 0.3
ngrahamUnsubmitted
Done

If we're emulating the shadows in D16968, shouldn't this be 0.35?

ngraham: If we're emulating the shadows in D16968, shouldn't this be 0.35?
rootyAuthorUnsubmitted
Done

Yes, but doesn't it look too jagged when it's 0.35?

0.35:


0.25:

rooty: Yes, but doesn't it look too jagged when it's 0.35? 0.35: {F6639986} 0.25: {F6639988}
45 color: ColorScope.backgroundColor 45​ color: "black" // matches Breeze window decoration and desktopcontainment
ngrahamUnsubmitted
Done

"New" isn't going to be accurate for very long. :)

We should also explain in the comment why we're hardcoding a color since this reflects a reversion of that change (continuity with Breeze shadow color, black looks better than gray, not even necessary to use a theme color since the text color is always white, etc)

ngraham: "New" isn't going to be accurate for very long. :) We should also explain in the comment why…
46 } 46​ }
47 47​
48 ColumnLayout { 48​ ColumnLayout {
49 id: clock 49​ id: clock
50 Label { 50​ Label {
51 text: Qt.formatTime(timeSource.data["Local"]["DateTime"]) 51​ text: Qt.formatTime(timeSource.data["Local"]["DateTime"])
52 style: softwareRendering ? Text.Outline : undefined 52​ style: softwareRendering ? Text.Outline : undefined
53 styleColor: softwareRendering ? ColorScope.backgroundColor : undefined 53​ styleColor: softwareRendering ? ColorScope.backgroundColor : undefined
Show All 18 Lines