-
-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update mine rebuilder userscript #386
Conversation
@PiTi2k5 you approve this? |
Without combobox on each deposit, the create of templates are useless or not? |
@PiTi2k5 and I had a quick chat, and we agreed to add a radio button in the header that will show all checkboxes or only the available ones. Let's put this PR on hold till then. |
The update has been pushed. |
i see u using "game" variable for storing DepositDepletedGameScriptKeys.SWITCH_STATUS.. why? its not saving to settings ang "game" is class variable..
|
It just stores the radio button's state while the client is running. So, if you close the modal and reopen it, the radio button's state will be restored. But I don't think we need to save this setting permanently in a file. I know the game is the main object, but the set key-value won't conflict with anything because it has a unique prefix. |
not necessary to escape from script scope.. u can use any variable defined inside script |
…h state temporarily
Ok, I've pushed the update for this. Using a global variable within the script instead of the |
This is an update for @PiTi2k5 's
user_mine_rebuild.js
userscript.We can now switch between 2 options: