hein added a comment.

  I'm not a fan of the combined icon idea. On face of it it's a neat "go the 
extra mile" kind of thing, on the other hand it creates follow-up which I'm 
certain will not be done in a timely manner, or even forgotten about: Updating 
those assets whenever the upstream branding is revised. It means initially the 
visual is slightly nicer, but in the future it will look outdated and crap. 
It's not a good idea to bring this about for such a minor thing, I think it's 
better to keep it simple and agree with Vlad here.

REPOSITORY
  R856 Plasma Browser Integration

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

To: hein, davidedmundson, apol, mart, #plasma
Cc: alexeymin, zzag, hein, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to