Code review comment for lp:~codygarver/elementaryos/seeds-add-notifyosd-theme

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

I'd use "notify-osd-icons-elementary" package name because the original package is called "notify-osd-icons". I'd definitely put it in artwork where the rest of the icons is. I'd also add "Suggests: notify-osd-icons-elementary" to elementary-icon-theme package.

review: Abstain

« Back to merge proposal