Code review comment for lp:~michael.rawson/unity/fix-show-desktop

Revision history for this message
Marco Trevisan (TreviƱo) (3v1n0) wrote :

Cool, one thing you could do instead, to avoid unneeded data duplication is to remove any reference to desktop_launcher_icon_ from LauncherController and to only use desktop_icon_ for everything.

Applying this should be enough:

Sorry for asking you to change this again, but your effort is appreciated :)


« Back to merge proposal