Code review comment for lp:~jjardon/libappindicator/fix-820080

Revision history for this message
Ted Gould (ted) wrote :

Sorry I took so long on this. We can't drop adding the "-panel" name extension on the icons as we want the fallback to look the same as the icon would look in the primary case. The "-panel" extension is used so that icon themes can theme the panel differently as they're not given anything like a StyleContext to determine their location otherwise.

review: Disapprove

« Back to merge proposal