cblack accepted this revision.
cblack added a comment.
This revision is now accepted and ready to land.


  Visually, this looks good.
  
  The code also looks makes sense and looks good to me, albeit there's some 
minor style issues, mostly in how properties are grouped and ordered.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  digital-clock-layouts-and-timezones-in-popup (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27903

To: ngraham, #vdg, #plasma, cblack
Cc: cblack, apol, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, 
zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, ahiemstra, mart

Reply via email to