Firstly, the auto-barrel (data-updates.lua creates a filled barrel item, fill and empty recipes, and assigns icons to them all) icon generation for the fill and empty recipes doesn't take into account icons with a size other than 32.
https://www.dropbox.com/s/qss8qzvvwef7h ... 4.png?dl=0
in this example screenshot, pure water is icon_size 64.
Also, if a fluid doesn't have an icon= tag (EG, it has an icons= tag), the function refuses to run on it at all.
Auto-barrel doesn't take into account icons with size > 32
- bobingabout
- Smart Inserter
- Posts: 7352
- Joined: Fri May 09, 2014 1:01 pm
- Contact:
Re: Auto-barrel doesn't take into account icons with size > 32
Thanks for fixing it for 0.17.43
- eradicator
- Smart Inserter
- Posts: 5207
- Joined: Tue Jul 12, 2016 9:03 am
- Contact:
Re: Auto-barrel doesn't take into account icons with size > 32
Btw, barreling also doesn't account for fluids with localised_name like it could:
Code: Select all
localised_name = {"recipe-name.fill-barrel", fluid.localised_name or {"fluid-name." .. fluid.name}},
Author of: Belt Planner, Hand Crank Generator, Screenshot Maker, /sudo and more.
Mod support languages: 日本語, Deutsch, English
My code in the post above is dedicated to the public domain under CC0.
Mod support languages: 日本語, Deutsch, English
My code in the post above is dedicated to the public domain under CC0.
- bobingabout
- Smart Inserter
- Posts: 7352
- Joined: Fri May 09, 2014 1:01 pm
- Contact:
Re: Auto-barrel doesn't take into account icons with size > 32
because if no localised_name is specified, that will return nil and use the or condition to use the default.eradicator wrote: ↑Thu May 23, 2019 12:55 pm Btw, barreling also doesn't account for fluids with localised_name like it could:Code: Select all
localised_name = {"recipe-name.fill-barrel", fluid.localised_name or {"fluid-name." .. fluid.name}},
simple fix.
is it localised or localized? there's a lot of American in the code.
- bobingabout
- Smart Inserter
- Posts: 7352
- Joined: Fri May 09, 2014 1:01 pm
- Contact:
Re: Auto-barrel doesn't take into account icons with size > 32
you should probably make a new bug report, as this one is in the resolved bin, and further conversation will likely be ignored.eradicator wrote: ↑Thu May 23, 2019 12:55 pm Btw, barreling also doesn't account for fluids with localised_name like it could:Code: Select all
localised_name = {"recipe-name.fill-barrel", fluid.localised_name or {"fluid-name." .. fluid.name}},
- eradicator
- Smart Inserter
- Posts: 5207
- Joined: Tue Jul 12, 2016 9:03 am
- Contact:
Re: Auto-barrel doesn't take into account icons with size > 32
Author of: Belt Planner, Hand Crank Generator, Screenshot Maker, /sudo and more.
Mod support languages: 日本語, Deutsch, English
My code in the post above is dedicated to the public domain under CC0.
Mod support languages: 日本語, Deutsch, English
My code in the post above is dedicated to the public domain under CC0.