- Issue created by @Anybody
- 🇩🇪Germany Anybody Porta Westfalica
This might be a nice novice task to learn JS refactoring to
core/once
and implementing tests. - 🇸🇮Slovenia miroslavstankov
Attached patch replaces the processed class pattern with core/once API as requested.
Changes made:
- Replaced hasClass/addClass('entity-browser-processed') with once() API in entityBrowserModal behavior
- Added 'once' as function parameter following the same pattern as entity_browser.tabs.js
- entity_browser.tabs.js was already correctly using once() - no changes neededTesting completed:
- All entity browser modals open and function correctly
- Entity selection works properly
- No JavaScript console errorsReady for review.
- 🇩🇪Germany Anybody Porta Westfalica
@miroslavstankov please use the MR, not patches.
- Merge request !102Issue #3542178: Replace processed class pattern with core/once API → (Open) created by Unnamed author
- 🇸🇮Slovenia miroslavstankov
Thanks for the feedback. I’ve created MR !102 to replace the patch, as requested. Ready for review.
- 🇸🇮Slovenia miroslavstankov
Renamed $button to $inputs and button to inputElement for clarity