Review Request 126851: Places data engine: Rename model role name "index" to "id"

2016-01-23 Thread Daniel Faust
t;Hide" don't seem to work anyway and the others were not tested. Thanks, Daniel Faust

Re: Review Request 126851: Places data engine: Rename model role name "index" to "id"

2016-01-23 Thread Daniel Faust
-- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126851/#review91476 ------- On Jan. 23, 2016, 2:14 p.m., Daniel Faust wrote: > > ---

Re: Review Request 126851: Places data engine: Rename model role name "index" to "id"

2016-01-23 Thread Daniel Faust
" and "Teardown Device" operations of the service. The operations "Show" and "Hide" don't seem to work anyway and the others were not tested. Thanks, Daniel Faust

Re: Review Request 126851: Places data engine: Rename model role name "index" to "id"

2016-01-27 Thread Daniel Faust
marked as submitted. Review request for kde-workspace and Plasma. Changes --- Submitted with commit 8bedfde2894a60dc72a0bd40589dfb900776dace by Daniel Faust to branch master. Repository: plasma-workspace Description --- I wrote a qml applet using the places data engine and noticed

Review Request 127102: Use fixed width for digital clock applet

2016-02-17 Thread Daniel Faust
displaying different combinations of "seconds", "date" and "timezone" works. Thanks, Daniel Faust

Review Request 127104: Don't show icon in notification area when checking for updates

2016-02-17 Thread Daniel Faust
available. Diffs - src/plasma/contents/ui/main.qml a847f45 Diff: https://git.reviewboard.kde.org/r/127104/diff/ Testing --- Thanks, Daniel Faust

Re: Review Request 127102: Use fixed width for digital clock applet

2016-02-18 Thread Daniel Faust
test and maximumCharacterWidth returned about twice the width actually needed. - Daniel --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127102/#review92514 ---------

Re: Review Request 127102: Use fixed width for digital clock applet

2016-02-18 Thread Daniel Faust
ontMetric's maximumCharacterWidth > > > > * numChars. > > > > Then we could kill sizeHelper competely (FontMetric's didn't exist when > > this was written) > > Marco Martin wrote: > hoping maximumCharacterWidth is reliable fo

Re: Review Request 127102: Use fixed width for digital clock applet

2016-02-18 Thread Daniel Faust
ontMetric's maximumCharacterWidth > > > > * numChars. > > > > Then we could kill sizeHelper competely (FontMetric's didn't exist when > > this was written) > > Marco Martin wrote: > hoping maximumCharacterWidth is reliable fo

Re: Review Request 127102: Use fixed width for digital clock applet

2016-02-19 Thread Daniel Faust
ontMetric's maximumCharacterWidth > > > > * numChars. > > > > Then we could kill sizeHelper competely (FontMetric's didn't exist when > > this was written) > > Marco Martin wrote: > hoping maximumCharacterWidth is reliable fo

Re: Review Request 127102: Use fixed width for digital clock applet

2016-02-19 Thread Daniel Faust
ontMetric's maximumCharacterWidth > > > > * numChars. > > > > Then we could kill sizeHelper competely (FontMetric's didn't exist when > > this was written) > > Marco Martin wrote: > hoping maximumCharacterWidth is reliable fo

Re: Review Request 127102: Use fixed width for digital clock applet

2016-02-19 Thread Daniel Faust
: https://git.reviewboard.kde.org/r/127102/diff/ Testing --- Works with horizontal and vertical panel. Also displaying different combinations of "seconds", "date" and "timezone" works. Thanks, Daniel Faust

Re: Review Request 127102: Use fixed width for digital clock applet

2016-02-26 Thread Daniel Faust
ontMetric's maximumCharacterWidth > > > > * numChars. > > > > Then we could kill sizeHelper competely (FontMetric's didn't exist when > > this was written) > > Marco Martin wrote: > hoping maximumCharacterWidth is reliable fo

Re: Review Request 127102: Use fixed width for digital clock applet

2016-03-08 Thread Daniel Faust
) - applets/digital-clock/package/contents/ui/DigitalClock.qml 95bb071 Diff: https://git.reviewboard.kde.org/r/127102/diff/ Testing --- Works with horizontal and vertical panel. Also displaying different combinations of "seconds", "date" and "timezone" works. Thanks, Daniel Faust

Re: Review Request 127102: Use fixed width for digital clock applet

2016-03-15 Thread Daniel Faust
thing. - Daniel --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127102/#review93561 --- On März 8, 2016, 6:53 nachm., Daniel F

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-03-18 Thread Daniel Faust
> On März 16, 2016, 6:19 nachm., Martin Klapetek wrote: > > applets/digital-clock/package/contents/ui/DigitalClock.qml, line 400 > > <https://git.reviewboard.kde.org/r/127393/diff/1/?file=453347#file453347line400> > > > > Have you tried dateLabelLeft.paint

Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-03-19 Thread Daniel Faust
iff: https://git.reviewboard.kde.org/r/127393/diff/ Testing --- Thanks, Daniel Faust

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-03-19 Thread Daniel Faust
- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127393/#review93620 ------- On März 16, 2016, 4:53 nachm., Daniel Faust wrote: > > --

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-03-19 Thread Daniel Faust
> On März 16, 2016, 6:19 nachm., Martin Klapetek wrote: > > applets/digital-clock/package/contents/ui/DigitalClock.qml, line 400 > > <https://git.reviewboard.kde.org/r/127393/diff/1/?file=453347#file453347line400> > > > > Have you tried dateLabelLeft.paint

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-03-19 Thread Daniel Faust
s (updated) - applets/digital-clock/package/contents/ui/DigitalClock.qml 02d55a9 Diff: https://git.reviewboard.kde.org/r/127393/diff/ Testing --- Thanks, Daniel Faust

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-03-19 Thread Daniel Faust
print an error that you can't and one is > > ignored > > > > Also leave this in the font { } format please > > Daniel Faust wrote: > Yes, I copy'n'pasted it. How about I throw in an additional commit to fix > it everywhere in the file?

Re: Review Request 127102: Use fixed width for digital clock applet

2016-03-19 Thread Daniel Faust
quot;A" and "P" width's and use that? Would spare > > creating two Date objects and two calls to Qt.formatTime > > Daniel Faust wrote: > No, because eg. in german the strings for am and pm are "vorm." and > "nachm.". > > Martin K

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-03-19 Thread Daniel Faust
nd use that as a scaling factor. - Daniel --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127393/#review93609 ------- On Mär

Re: Review Request 127102: Use fixed width for digital clock applet

2016-03-19 Thread Daniel Faust
0001-Use-fixed-width-for-digital-clock-applet.patch https://git.reviewboard.kde.org/media/uploaded/files/2016/03/16/81b4a902-1454-4155-9fda-552b8acba1a8__0001-Use-fixed-width-for-digital-clock-applet.patch Thanks, Daniel Faust

Re: Review Request 127102: Use fixed width for digital clock applet

2016-03-19 Thread Daniel Faust
marked as submitted. Review request for kde-workspace and Plasma. Changes --- Submitted with commit e7f09ba1eb976c4f282145016d34fe87de515a25 by Daniel Faust to branch master. Bugs: 347724 https://bugs.kde.org/show_bug.cgi?id=347724 Repository: plasma-workspace Description

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-03-21 Thread Daniel Faust
, visit: https://git.reviewboard.kde.org/r/127393/#review93821 --- On März 19, 2016, 2:16 nachm., Daniel Faust wrote: > > --- > This is an automatically generated e-mail. T

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-04-09 Thread Daniel Faust
> On März 21, 2016, 4:08 nachm., Martin Klapetek wrote: > > Thanks! > > Daniel Faust wrote: > I already started to rewrite the clock applet. I got pretty far actually, > and this issue arose again. > I think it was always intended to use the default font size

Re: Review Request 127393: digital-clock: Fix font size of date label in small panels

2016-04-09 Thread Daniel Faust
separator height to 70% of the applet height. Diffs - applets/digital-clock/package/contents/ui/DigitalClock.qml 02d55a9 Diff: https://git.reviewboard.kde.org/r/127393/diff/ Testing --- Thanks, Daniel Faust