Page MenuHomeWildfire Games

Attempt to update food icon
AbandonedPublic

Authored by Silier on Jan 9 2021, 9:10 PM.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

Currently we have action cursors showing subtypes of resources and we use global type in other cases.
Hovewer meat and food icons are identical. Also as preparation for D3310, it would be nice to have more generic food icon.

From my perspective new icon looks decent but It would be preferable if some artist would look into it.

Test Plan

Try in game and probably complain.

Event Timeline

Silier created this revision.Jan 9 2021, 9:10 PM
Owners added a subscriber: Restricted Owners Package.Jan 9 2021, 9:10 PM
Silier published this revision for review.Jan 9 2021, 9:11 PM
Vulcan added a comment.Jan 9 2021, 9:16 PM

Build is green

builderr-debug-macos.txt
ld: warning: text-based stub file /System/Library/Frameworks//CoreAudio.framework/CoreAudio.tbd and library file /System/Library/Frameworks//CoreAudio.framework/CoreAudio are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox.tbd and library file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback.tbd and library file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//CoreVideo.framework/CoreVideo.tbd and library file /System/Library/Frameworks//CoreVideo.framework/CoreVideo are out of sync. Falling back to library file for linking.
ld: warning: text-based stu

See https://jenkins.wildfiregames.com/job/macos-differential/2738/display/redirect for more details.

Nescio added a subscriber: Nescio.Jan 9 2021, 9:56 PM

The proposed icon might work as a placeholder but is in my opinion not good enough for a release.
What I did in my mod is take the art/textures/ui/session/portraits/technologies/grain.png, give it a transparent background, and use that as a placeholder: https://github.com/0abc/0abc-a23/tree/master/art/textures/ui/session/icons/resources :


I'm not saying you should do the same, but there are other images that might look at for inspiration (e.g. baskets or olives; the cornucopia is a bit too fancy, maize is native to the Americas).
You don't really need to limit yourself to 16×16, showing images at larger than their own size tends to look poorly, but displaying images at smaller than their own size usually works

Silier abandoned this revision.May 23 2021, 3:10 PM