Change Details

During the lead paragraph work we began a refactor of the MobileFormatter class for best testability and readability. Code coverage of the new transforms is good: https://doc.wikimedia.org/cover-extensions/MobileFrontend/transforms/index.html https://doc.wikimedia.org/cover-extensions/MobileFrontend/MobileFormatter.php.html = Work (incomplete) [X] Separate the LazyLoadedImages transform from MobileFormatter (https://gerrit.wikimedia.org/r/448415) [] Separate the LazyLoadedReferences transform from MobileFormatter [x] Separate the LegacyMainPage transform from MobileFormatter (https://gerrit.wikimedia.org/r/#/c/401786/) [] Separate out the RemoveImages transform from MobileFormatter [] Separate out the SectionWrapping transform from MobileFormatter [] Review remaining transforms and their purpose (are they redundant?) [x] Add tests for AddMobileTocTransform [] MobileContext contains many methods which should be migrated to the MobileFormatter - for example, [get|set]ContentTransformations. Review and port as appropriate

During the lead paragraph work we began a refactor of the MobileFormatter class for best testability and readability. Code coverage of the new transforms is good: https://doc.wikimedia.org/cover-extensions/MobileFrontend/transforms/index.html https://doc.wikimedia.org/cover-extensions/MobileFrontend/MobileFormatter.php.html = Work (incomplete) [X] Separate the LazyLoadedImages transform from MobileFormatter (https://gerrit.wikimedia.org/r/448415) [x] Separate the LegacyMainPage transform from MobileFormatter (https://gerrit.wikimedia.org/r/#/c/401786/) [x] Add tests for AddMobileTocTransform [] Separate out the RemoveImagesTransform from MobileFormatter and mark as @deprecated referencing T186627) [] Separate out the MobileSectionTransform transform from MobileFormatter # Sign off steps [] MobileContext contains many methods which should be migrated to the MobileFormatter - for example, [get|set]ContentTransformations. Review and port as appropriate. Create tasks. [] Review remaining transforms and their purpose (are they redundant?) . Create tasks.

During the lead paragraph work we began a refactor of the MobileFormatter class for best testability and readability. Code coverage of the new transforms is good: https://doc.wikimedia.org/cover-extensions/MobileFrontend/transforms/index.html https://doc.wikimedia.org/cover-extensions/MobileFrontend/MobileFormatter.php.html = Work (incomplete) [X] Separate the LazyLoadedImages transform from MobileFormatter (https://gerrit.wikimedia.org/r/448415) [] Separate the LazyLoadedReferences transform from MobileFormatter [x] Separate the LegacyMainPage transform from MobileFormatter (https://gerrit.wikimedia.org/r/#/c/401786/) [] Separate out the RemoveImages transform from MobileFormatter [] Separate out the SectionWrapping transform from MobileFormatterx] Add tests for AddMobileTocTransform [] Review remaining tSeparate out the RemoveImagesTransforms and their purpose (are they redundant? from MobileFormatter and mark as @deprecated referencing T186627) [x] Add tests for AddMobileTocTransform] Separate out the MobileSectionTransform transform from MobileFormatter # Sign off steps [] MobileContext contains many methods which should be migrated to the MobileFormatter - for example, [get|set]ContentTransformations. Review and port as appropriate. Create tasks. [] MobileContext contains many methods which should be migrated to the MobileFormatter - for example, [get|set]ContentTransformationsReview remaining transforms and their purpose (are they redundant?) . Review and port as appropriateCreate tasks.