[Differential] [Commented On] D2212: Reduce Calendar size in Digital clock

2016-07-20 Thread mck182 (Martin Klapetek)
mck182 added a comment. In https://phabricator.kde.org/D2212#41055, @drosca wrote: > The calendar popup size is fine for me (with patched plasma-framework), what I *still* think is broken is the gridSize scaling - see https://git.reviewboard.kde.org/r/125773/ > > You can also test it

[Differential] [Commented On] D2212: Reduce Calendar size in Digital clock

2016-07-18 Thread mck182 (Martin Klapetek)
mck182 added a comment. Can you post a screenshot of before/after? For future reference - any review request that touches the UI should always be accompanied by a screenshot ;) REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D2212 EMAIL PREF

[Differential] [Updated] D1478: [Notifications] Don't group job finished notifications

2016-04-24 Thread mck182 (Martin Klapetek)
mck182 added a comment. In https://phabricator.kde.org/D1478#27445, @colomar wrote: > Why are those notification even persistent in the first place? If a user misses the notification, they should notice that the job was finished by the fact that it's not running anymore. > Only errors

[Differential] [Accepted] D901: [KRunner] Use time-based sorting rather than length for auto-completion

2016-02-04 Thread mck182 (Martin Klapetek)
mck182 accepted this revision. mck182 added a comment. This revision is now accepted and ready to land. Well that was easy :) REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D901 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/email

[Differential] [Closed] D899: [notifications] Make the text selectable in the popup too

2016-02-04 Thread mck182 (Martin Klapetek)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMAWORKSPACE4819282d329f: [notifications] Make the text selectable in the popup too (authored by mck182). REPOSITORY rPLASMAWORKSPACE Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.

[Differential] [Updated, 128 lines] D899: [notifications] Make the text selectable in the popup too

2016-02-04 Thread mck182 (Martin Klapetek)
mck182 updated this revision to Diff 2192. mck182 added a comment. Keep the MouseArea visible always as it pushes the action buttons to the side REPOSITORY rPLASMAWORKSPACE Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D899?vs=2190&id=2192 BRANCH mklapetek/sele

[Differential] [Updated] D899: [notifications] Make the text selectable in the popup too

2016-02-04 Thread mck182 (Martin Klapetek)
mck182 added a reviewer: Plasma. REPOSITORY rPLASMAWORKSPACE Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D899 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mck182, Plasma Cc: plasma-devel ___

[Differential] [Request, 128 lines] D899: [notifications] Make the text selectable in the popup too

2016-02-04 Thread mck182 (Martin Klapetek)
mck182 created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. REVISION SUMMARY This simplifies the component by removing the Loader-property combo and leaves just a read-only TextEdit BUG: 358981 REPOSITORY rPLASMAWORKSPACE Plasma Workspace BRA