Bug#887420: RFS: deepin-icon-theme/15.12.52-1 [ITP]
2018-01-17 5:18 GMT+08:00 Adam Borowski <kilobyte@angband.pl>:
> On Tue, Jan 16, 2018 at 04:52:04PM +0800, Yangfl wrote:
>> * Package name : deepin-icon-theme
>
>> It builds those binary packages:
>>
>> deepin-icon-theme - Deepin Icon Theme
>
> It provides both an icon and a cursor theme, it'd be good to mention this.
>
Done.
> Also, the priority for x-cursor-theme looks a wee bit inflated.
> I'm not sure what's a good value, though. The problem is, the priority is
> shared between all installed desktop environments, affecting the system
> globally -- while one would expect an environment to pick its own default
> unless overridden by the user.
>
Changed to 50.
>
> 喵!
> --
> ⢀⣴⠾⠻⢶⣦⠀
> ⣾⠁⢰⠒⠀⣿⡁ Imagine there are bandits in your house, your kid is bleeding out,
> ⢿⡄⠘⠷⠚⠋⠀ the house is on fire, and seven big-ass trumpets are playing in the
> ⠈⠳⣄⠀⠀⠀⠀ sky. Your cat demands food. The priority should be obvious...
Please review it again, thanks a lot.
Sincerely,
Reply to: