On Wed, Oct 24, 2018 at 12:34:20PM +0300, Dmitry Timoshkov wrote:
Hi Huw,
Huw Davies huw@codeweavers.com wrote:
@@ -427,6 +427,9 @@ FIXME: Need to add them, but for now just let them use the same: searching.avi /* explorer toolbar icons
- FIXME: images are hacky and should be re-drawn; also dark and light bitmaps are same for now
*/
I'd suggest moving the new defines before the above comment, since the comment relates to the toolbar bitmaps. Better still, perhaps the comment should just go away?
The bimaps I've copied from user32 also need to be redrawn since they should look a bit differently, so I'd prefer to leave both the comment and the bitmaps order.
They're not 'explorer toolbar icons', right?
Huw.