Code review comment for lp:~zeitgeist/activity-log-manager/smaller-exclude-icons

Revision history for this message
Rico Tzschichholz (ricotz) wrote :

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)

review: Disapprove

« Back to merge proposal