This revision was automatically updated to reflect the committed changes.
Closed by commit R266:dca95f680776: New icon for KTimeTracker (authored by
cblack).
REPOSITORY
R266 Breeze Icons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D26441?vs=74293=75263
REVISION DETAIL
ndavis accepted this revision.
ndavis added a comment.
This revision is now accepted and ready to land.
I think I'll just accept this as it is.
REPOSITORY
R266 Breeze Icons
BRANCH
ktimetracker-icon (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D26441
To: cblack,
cblack added a comment.
@ndavis ping
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D26441
To: cblack, #vdg, ndavis
Cc: hellcp, ndavis, kde-frameworks-devel, aspotashev, LeGast00n, cblack, GB_2,
michaelh, ngraham, bruns
cblack edited the test plan for this revision.
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D26441
To: cblack, #vdg, ndavis
Cc: hellcp, ndavis, kde-frameworks-devel, aspotashev, LeGast00n, GB_2,
michaelh, ngraham, bruns
cblack updated this revision to Diff 74293.
cblack added a comment.
Use a rich grey instead of a rich mahogany
REPOSITORY
R266 Breeze Icons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D26441?vs=74286=74293
BRANCH
ktimetracker-icon (branched from master)
REVISION DETAIL
ndavis requested changes to this revision.
ndavis added a subscriber: hellcp.
ndavis added a comment.
This revision now requires changes to proceed.
I have to agree with what @hellcp said in the VDG chat earlier, the mahogany
really does not fit. It draws attention to the wrong places and is
cblack updated this revision to Diff 74286.
cblack added a comment.
Adjust colour from blue to mahogany
REPOSITORY
R266 Breeze Icons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D26441?vs=72823=74286
BRANCH
ktimetracker-icon (branched from master)
REVISION DETAIL
ndavis added a comment.
The top part looks a bit too flat and wide. There's also nothing attaching it
to the body, so it looks like it's floating.
I don't think Plasma Blue work well as a body color with no outline. This
happens when I select the file in dolphin because it blends in
cblack edited the test plan for this revision.
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D26441
To: cblack, #vdg
Cc: kde-frameworks-devel, aspotashev, LeGast00n, GB_2, michaelh, ngraham, bruns
cblack created this revision.
cblack added a reviewer: VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
cblack requested review of this revision.
REVISION SUMMARY
This diff adds a new icon for KTimeTracker.
T12305
10 matches
Mail list logo
