D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Noah Davis
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit R266:3e095e86bfae: Add call-incoming/missed/outgoing (authored by ndavis). REPOSITORY R266 Breeze Icons CHANGES SINCE

D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Noah Davis
ndavis added a comment. In D28114#629728 , @broulik wrote: > Leave the phone in, please. I'm having a hard time grasping the pure arrow icons Android uses in the dialer. I knew I couldn't be the only one :) REPOSITORY R266 Breeze

D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Kai Uwe Broulik
broulik added a comment. Leave the phone in, please. I'm having a hard time grasping the pure arrow icons Android uses in the dialer. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D28114 To: ndavis, #vdg, cblack Cc: broulik, cblack, jbbgameich, bshah,

D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Noah Davis
ndavis edited the test plan for this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D28114 To: ndavis, #vdg, cblack Cc: cblack, jbbgameich, bshah, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Noah Davis
ndavis updated this revision to Diff 77889. ndavis added a comment. - Move to status category - Make phone size better and more consistent at 16px REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28114?vs=77877=77889 BRANCH call-history (branched

D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Carson Black
cblack requested changes to this revision. cblack added a comment. This revision now requires changes to proceed. The 16px icons should probably drop the phone, as the distinguishing elements between them are so small as to impair the ease of telling the icons apart. Legibility should be

D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Bhushan Shah
bshah added a subscriber: jbbgameich. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D28114 To: ndavis, #vdg Cc: jbbgameich, bshah, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Noah Davis
ndavis edited the summary of this revision. ndavis edited the test plan for this revision. ndavis added a subscriber: bshah. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D28114 To: ndavis, #vdg Cc: bshah, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh,

D28114: Add call-incoming/missed/outgoing

2020-03-18 Thread Noah Davis
ndavis created this revision. ndavis added a reviewer: VDG. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. ndavis requested review of this revision. REPOSITORY R266 Breeze Icons BRANCH call-history (branched from master) REVISION DETAIL