MobileFrontendSkinHooks::interimTogglingSupport() should probably be a public method on includes/Transforms/MakeSectionsTransform.php since it relates to that code and gradeCImageSupport() should be on includes/Transforms/LazyImageTransform.php.
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Refactor: Move `Transforms` related code to correct transform classes | mediawiki/extensions/MobileFrontend | master | +117 -113 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T263879 Suggest renaming "MobileFrontendSkinHooks.php" to "MobileFrontendSkin.php" or something better | |||
| Resolved | Jdlrobson | T264155 Move Transforms related code from MobileFrontendSkinHooks.php |
Event Timeline
Comment Actions
Change 631131 had a related patch set uploaded (by D3r1ck01; owner: Derick A):
[mediawiki/extensions/MobileFrontend@master] Refactor: Move Transforms related code to correct Translform classes
Comment Actions
@Jdlrobson, once this gets reviewed & hopefully merged, I'll work on the next refactor task. Thanks!
Comment Actions
Change 631131 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Refactor: Move Transforms related code to correct transform classes