Add settings

[Athozus]

This is a merge request, according to #51, to add settings to the mod. Currently there are not the sounds/trash settings like I did on the issue, cause it's not yet implemented (will be in few days).

Some interesting infos :

• It is in the menu "Options" which replace the About button, and this menu has two tabs : Settings and About, maybe Help in the future.
• It uses an entry.settings with following syntax :

{
name = "setting",
value = "itsvalue", -- could be boolean, int or string
}

Closes #51

[Athozus]

Quick screenshot

[BuckarooBanzay]

didn't test but looks good 👍 (thanks for addressing the table/list thing)

[Athozus]

I have a quick issue reported by @Bastrabun. On some servers, when you press escape, it is still there. It is due to the mail.show_settings() at the end of the events checking, I would like to remove it, but when I move it into if fields.reset you have to change of formspec and go again to settings to see the settings reseted. So, I would like to know if there is an event like if escape_pressed, to avoid that weird behaviour.

[BuckarooBanzay]

you could use the button_exit[] or an explicit "save"-field to check for, but other than that, no clue sorry :/

[Athozus]

Yeah at first I would like to do that but I think that checkboxes can't give their values when there aren't toggled.

[S-S-X]

LGTM but also didn't actually test anything.

[Athozus]

LGTM but also didn't actually test anything.

Nope, I have to find a way to get checkboxes values while pressing Save (not the event related to the checkbox itself).

Review again when ready for review.

[Athozus]

This one is fixed, I merge so (I used selected_idxs finally :D).