Module talk:Message box/configuration
Protected edit request on 22 January 2014
 | This edit request has been answered. Set the |answered= parameter to no to reactivate your request. |
Please make the edit to the live version of this module that I made in the sandbox to allow amboxs to have a "hidden" parameter (this is for the {{Orphan}} revamp that will allow it to be hidden when alone on a page and visible when inside a multiple issue) and have it be empty as-well-as to allow tmboxs to have custom id attributes passed (to allow me to complete my User:Technical 13/SandBox/help-helper-tools.js script (which I already want to adapt to use for assisting in answering EXp and EP requests once completed)). Thank you. Technical 13 (talk) 20:56, 22 January 2014 (UTC)
- Question – Is it possible to test the use of this new Message box/configuration in {{Orphan/sandbox}} and {{Orphan/testcases}} before going live with it? Wbm1058 (talk) 13:58, 23 January 2014 (UTC)
- I'm pretty sure it has been tested enough. The sandbox configuration module has been used in the sandbox module, and according to Mr. Stradivarius was working properly. As far as the second part of the request to allow tmboxs to have custom id attributes passed (to allow me to complete my User:Technical 13/SandBox/help-helper-tools.js script (which I already want to adapt to use for assisting in answering EXp and EP requests once completed)), I've slept on it and decided to use a class since there may be more than one tmbox on a page using the same id (or class name) and using class would be more correct. Technical 13 (talk) 15:22, 23 January 2014 (UTC)
- I don't think I ever tested the new config file settings specifically, so best not to rely on that before going live. I'll try and have a look later on to see if things look ok. — Mr. Stradivarius ♪ talk ♪ 21:48, 23 January 2014 (UTC)
- Also, I just found an old bug in the main module, so that should be fixed at the same time we roll this feature out. (The fix is in the sandbox.) The bug only affects calls from other Lua modules, and only when there is more than one message box created from the same module, so it's not so urgent that we need to fix it right this minute. I'll review Technical 13's changes and update both the module and the config file soon. — Mr. Stradivarius ♪ talk ♪ 06:59, 24 January 2014 (UTC)
Done. Sorry for the delay. Let me know if you see any odd behaviour from the module. — Mr. Stradivarius ♪ talk ♪ 01:08, 27 January 2014 (UTC)
- Also, I just found an old bug in the main module, so that should be fixed at the same time we roll this feature out. (The fix is in the sandbox.) The bug only affects calls from other Lua modules, and only when there is more than one message box created from the same module, so it's not so urgent that we need to fix it right this minute. I'll review Technical 13's changes and update both the module and the config file soon. — Mr. Stradivarius ♪ talk ♪ 06:59, 24 January 2014 (UTC)
- I don't think I ever tested the new config file settings specifically, so best not to rely on that before going live. I'll try and have a look later on to see if things look ok. — Mr. Stradivarius ♪ talk ♪ 21:48, 23 January 2014 (UTC)
- I'm pretty sure it has been tested enough. The sandbox configuration module has been used in the sandbox module, and according to Mr. Stradivarius was working properly. As far as the second part of the request to allow tmboxs to have custom id attributes passed (to allow me to complete my User:Technical 13/SandBox/help-helper-tools.js script (which I already want to adapt to use for assisting in answering EXp and EP requests once completed)), I've slept on it and decided to use a class since there may be more than one tmbox on a page using the same id (or class name) and using class would be more correct. Technical 13 (talk) 15:22, 23 January 2014 (UTC)