Request: Upgrade of Smart Display Mod to 0.17

Place to get help with not working mods / modding interface.
Post Reply
Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

Greetings,

I was wondering if anyone could upgrade the Smart Display mod from 0.16 to 0.17.
I've been waiting (and even tried and failed to do it myself) for someone to do it but nothing.

Here's the URL: https://mods.factorio.com/mod/SmartDisplayRedux_Fixed

Thanks for such a great game! :)

User avatar
darkfrei
Smart Inserter
Smart Inserter
Posts: 2903
Joined: Thu Nov 20, 2014 11:11 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by darkfrei »

Pixel_Builder wrote:
Wed May 22, 2019 11:44 pm
I was wondering if anyone could upgrade the Smart Display mod from 0.16 to 0.17.
Did you try just change the game version in the info.json?

Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

Yes - But that failed as well.

Here's the error msg I get when loading:
Attachments
Smart_Display_Err.jpg
Smart_Display_Err.jpg (57.31 KiB) Viewed 2536 times

User avatar
darkfrei
Smart Inserter
Smart Inserter
Posts: 2903
Joined: Thu Nov 20, 2014 11:11 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by darkfrei »

You are need to find and replace "textfield_style" to "textfield"

eduran
Filter Inserter
Filter Inserter
Posts: 344
Joined: Fri May 09, 2014 2:52 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by eduran »

darkfrei wrote:
Thu May 23, 2019 1:38 pm
You are need to find and replace "textfield_style" to "textfield"
That is not correct, you need to replace every instance of "textfield_style" with "textbox_style":

Code: Select all

type = "textbox_style"  -- good
type = "textfield_style"  --bad
There were a lot of changes to GUI elements, so I would expect further errors after this one.

Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

Thank you so much for your assistance. :D
That fixed that error but you are correct - It led to another.
Attachments
Smart_Display_Err_2.jpg
Smart_Display_Err_2.jpg (44.32 KiB) Viewed 2514 times

eduran
Filter Inserter
Filter Inserter
Posts: 344
Joined: Fri May 09, 2014 2:52 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by eduran »

Sprite width and height can no longer be zero in 0.17. You need to replace all instances of "height = 0" and "width = 0" with "height = 1" and "width = 1".

Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

:) Which now produces this...
Attachments
Smart_Display_Err_3.jpg
Smart_Display_Err_3.jpg (35.84 KiB) Viewed 2504 times

eduran
Filter Inserter
Filter Inserter
Posts: 344
Joined: Fri May 09, 2014 2:52 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by eduran »

That's a 0.17 classic :D Delete every line that says

Code: Select all

flags = {"goes-to-quickbar"}

Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

OK - I was able to fix that and I was also able to fix the Research Tech packs but now I get this:
Attachments
Smart_Display_Err_6.jpg
Smart_Display_Err_6.jpg (49.08 KiB) Viewed 2498 times

eduran
Filter Inserter
Filter Inserter
Posts: 344
Joined: Fri May 09, 2014 2:52 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by eduran »

A bit more complicated. In prototypes/display.lua find

Code: Select all

if invisible then
  -- invisible.additional_pastable_entities = {"smart-display"}
  invisible.minable.result = "smart-display-visible"
  invisible.picture_off.filename = "__SmartDisplayRedux_Fixed__/graphics/empty.png"
  invisible.picture_off.width = 0
  invisible.picture_off.height = 0
  invisible.picture_on.filename = "__SmartDisplayRedux_Fixed__/graphics/empty.png"
  invisible.picture_on.width = 0
  invisible.picture_on.height = 0
end
and replace it with

Code: Select all

if invisible then
  invisible.minable.result = "smart-display-visible"
  invisible.picture_off = {
    filename = "__core__/graphics/empty.png",
    width = 1,
    height = 1,
  }
  invisible.picture_on = {
    filename = "__core__/graphics/empty.png",
    width = 1,
    height = 1,
  }
end

User avatar
darkfrei
Smart Inserter
Smart Inserter
Posts: 2903
Joined: Thu Nov 20, 2014 11:11 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by darkfrei »

The prototype with the "empty.png" must be 1x1, not 1x2 as yo made

Code: Select all

filename = "__SmartDisplayRedux_Fixed__/graphics/empty.png"
width = 1
height = 1
Or just change this picture to 2x2, it can help too.

Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

OK - So now I'm getting this:
Attachments
Smart_Display_Err_8.jpg
Smart_Display_Err_8.jpg (43.35 KiB) Viewed 2486 times

eduran
Filter Inserter
Filter Inserter
Posts: 344
Joined: Fri May 09, 2014 2:52 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by eduran »

You must have missed one "width = 0".

Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

OK - Thanks for your help.


Maybe someone more knowledgeable in this can take a look because it's beyond me.

User avatar
darkfrei
Smart Inserter
Smart Inserter
Posts: 2903
Joined: Thu Nov 20, 2014 11:11 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by darkfrei »

Pixel_Builder wrote:
Thu May 23, 2019 8:57 pm
Maybe someone more knowledgeable in this can take a look because it's beyond me.
Place your work here and I can fix the rest, it's not so difficult as you think.

Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

Great - Here it is. I've done what is on this page. Thank you.
Attachments
SmartDisplayRedux_Fixed_1.2.2.zip
(66.36 KiB) Downloaded 38 times

eduran
Filter Inserter
Filter Inserter
Posts: 344
Joined: Fri May 09, 2014 2:52 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by eduran »

This version loads, but I can't promise it will work properly.
Attachments
SmartDisplayRedux_Fixed_1.2.2.zip
(63.28 KiB) Downloaded 41 times

Pixel_Builder
Burner Inserter
Burner Inserter
Posts: 9
Joined: Mon Mar 07, 2016 3:00 pm
Contact:

Re: Request: Upgrade of Smart Display Mod to 0.17

Post by Pixel_Builder »

Good Morning,

Thank you eduran for all your help. It did load as expected and I'll report back as soon as I test it.

Update 1: Works as it should!!!

Thank you so much for your help again and enjoy your weekend! :D
Attachments
Working_Display.jpg
Working_Display.jpg (124.19 KiB) Viewed 2420 times

Post Reply

Return to “Modding help”