ndavis added a comment.
In D23464#522459 <https://phabricator.kde.org/D23464#522459>, @alexde wrote: > In D23464#522270 <https://phabricator.kde.org/D23464#522270>, @trickyricky26 wrote: > > > As @noahdvs mentioned above, the white background should be removed and the entire symbol should be made up of one filled path with no outline (!). This is important to keep the icon looking as intended with the Qt SVG renderer > > > He mentioned that the white background should be removed. He did **not** mention, that it should be made up of one filled path with no outline. ;-) > > > As for the 32px and 64px designs, I would much prefer using only two colors by placing white variants of the smaller icons on top of a blue document background. > > I think this fits much better with other Breeze mimetype icons. > > ndavis saw this differently: > > In D23464#521662 <https://phabricator.kde.org/D23464#521662>, @ndavis wrote: > > > I think we should use `#1ed4e5` for the color instead since that's what the official logo uses. Normally, I'd say to use a similar Breeze-like color, but mimetype icons are a mess anyway and I don't think it's worth worrying too much about unless Breeze's official color palette is expanded. > > > If we go with your suggestion, I think the outer ring should be dropped completely as it distracts more from the root / original branding than it benefits IMHO, like that: F7296477: Screenshot_20190830_155555.png <https://phabricator.kde.org/F7296477> > > It looks "Breezie" to me. :) Alright, try it without the ring. I thought the ring was necessary for it to make the original branding. Removing the ring allows the "root" symbol in the middle to be larger, which makes it less faint. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D23464 To: alexde, #vdg, ndavis, ngraham Cc: trickyricky26, ndavis, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
