D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-12-06 Thread Simon Peter
probono added a comment. Thank you @kossebau, really nice to see this. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17086 To: kossebau, #vdg, ngraham, TheAssassin Cc: probono, TheAssassin, ngraham, kde-frameworks-devel, michaelh, bruns

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-27 Thread TheAssassin
TheAssassin added a comment. Yep, it's really nice, thanks @kossebau! REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17086 To: kossebau, #vdg, ngraham, TheAssassin Cc: TheAssassin, ngraham, kde-frameworks-devel, michaelh, bruns

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-27 Thread Nathaniel Graham
ngraham added a comment. Thanks for the nice new icon! REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17086 To: kossebau, #vdg, ngraham, TheAssassin Cc: TheAssassin, ngraham, kde-frameworks-devel, michaelh, bruns

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-27 Thread Friedrich W. H. Kossebau
This revision was automatically updated to reflect the committed changes. Closed by commit R266:e3266db59d2f: Add icons for application-vnd.appimage/x-iso9660-appimage (authored by kossebau). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D17086?vs=46231&i

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-27 Thread Friedrich W. H. Kossebau
kossebau added a comment. Thanks for feedback/review :) REPOSITORY R266 Breeze Icons BRANCH addappimageiconv2 REVISION DETAIL https://phabricator.kde.org/D17086 To: kossebau, #vdg, ngraham, TheAssassin Cc: TheAssassin, ngraham, kde-frameworks-devel, michaelh, bruns

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-27 Thread TheAssassin
TheAssassin accepted this revision. REPOSITORY R266 Breeze Icons BRANCH addappimageiconv2 REVISION DETAIL https://phabricator.kde.org/D17086 To: kossebau, #vdg, ngraham, TheAssassin Cc: TheAssassin, ngraham, kde-frameworks-devel, michaelh, bruns

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-26 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Yep, that was it. Looks beautiful. Ship it! REPOSITORY R266 Breeze Icons BRANCH addappimageiconv2 REVISION DETAIL https://phabricator.kde.org/D17086 To: kossebau, #vdg, ngraham Cc

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-25 Thread Friedrich W. H. Kossebau
kossebau added a comment. In D17086#366120 , @ngraham wrote: > Looks great to me then! > > But how do I make this work? I compiled and installed the icon and verified that it's installed in the right place. But my AppImage isn't getting the n

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-25 Thread Nathaniel Graham
ngraham added a comment. Looks great to me then! But how do I make this work? I compiled and installed the icon and verified that it's installed in the right place. But my AppImage isn't getting the new icon, not even after running `update-desktop-database`. F6441744: Weird.png

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-25 Thread Friedrich W. H. Kossebau
kossebau added a comment. In D17086#366111 , @ngraham wrote: > Thanks, that looks great! As for the background color, I'd say it's okay to use a different shade of blue if it would better match the AppImage branding. For now I favor breez

D17086: Add icons for application-vnd.appimage/x-iso9660-appimage

2018-11-25 Thread Friedrich W. H. Kossebau
kossebau retitled this revision from "Add icons for application-vnd.appimage" to "Add icons for application-vnd.appimage/x-iso9660-appimage". REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D17086 To: kossebau, #vdg, ngraham Cc: TheAssassin, ngraham, kde-frameworks-