← Back to team overview

zeitgeist team mailing list archive

Re: [Merge] lp:~zeitgeist/activity-log-manager/smaller-exclude-icons into lp:activity-log-manager

 

Review: Disapprove

I don't see the need in reducing the icon-sizes. The problem which you want to fix is not caused by them. The underlying widget structure is statically sized and therefore doesn't adapt to different font-sizes.

And yes, all icons should have the same sizes.

So this approach is not the right thing to do imo. (which makes the bug invalid as well)
-- 
https://code.launchpad.net/~zeitgeist/activity-log-manager/smaller-exclude-icons/+merge/173678
Your team Zeitgeist Framework Team is subscribed to branch lp:~zeitgeist/activity-log-manager/smaller-exclude-icons.


References