Добавя в менюто на статии възможност за бързо преместване в инкубатора.
Details
Тествано в моя потребителски common.js.
Diff Detail
- Repository
- rUI JS/CSS User Interface
- Branch
- enh/move-to-incubator
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 24 Build 24: arc lint + arc unit
Event Timeline
Gadget-MoveToIncubator.js | ||
---|---|---|
47 | Последната анонимна функция е излишна, защото променливите $ и mw вече са налични в глобалното именно пространство. Достатъчен е реда: if ( mw.config.get( 'wgNamespaceNumber' ) === 0 ) initMoveToIncubatorMenu(); |
Gadget-MoveToIncubator.js | ||
---|---|---|
47 | Да, прав си. В твоя първоначален вариант функцията се изпълняваше веднага, а не при $(document).ready(), така че явно няма нужда от допълнителната обвивка. Въпреки това е по-добре да се обвие с ready(): if ( mw.config.get( 'wgNamespaceNumber' ) === 0 ) $( initMoveToIncubatorMenu ); |
Developing with ResourceLoader
While here, also set the parentheses-space style to jQ everywhere.
Gadget-MoveToIncubator.js | ||
---|---|---|
47 | @Borislav Мда, признавам, че вчера малко претупах нещата с копи-пейст, пък и съм позабравил доста работи. Отделно колко поначало ги знам :D Погледнах пак примерите в Developing with ResourceLoader и върнах анонимната функция, но вече като биндната към ready. Не знам дали има практическа разлика дали ще бъде бинднат целия if, или само извикването на initMoveToIncubatorMenu(), но така ми изглежда някак по-прегледно. Дано не пропускам нещо. :) |