Change Details

Many projects have been using parsoid now. While doing some code cleanup in MobileFrontend I noticed Kartographer makes use of the mobileFrontend.section-toggled hook which is not present in the Parsoid version and in the legacy parser marked as `Internal for use inside ExternalGuidance.` https://gerrit.wikimedia.org/g/mediawiki/extensions/Kartographer/+/2914baaa1260d7cf3cf9961fcb3e7503c028478e/modules/mapframe/mapframe.js#102 Is this still needed? If so could you provide context on what bug it is addressing so the #readers-growth team can consider whether to ? # TODO [] Determine if Kartographer still needs it (And if there is a bug in production that has resurfaced relating to this) [] If needed, update the documentation that Kartographer uses the hook and add it to the Parsoid sectionCollapsing code. [] If not remove hook from code.

Many projects have been using parsoid now. While doing some code cleanup in MobileFrontend I noticed Kartographer makes use of the mobileFrontend.section-toggled hook which is not present in the Parsoid version and in the legacy parser marked as `Internal for use inside ExternalGuidance.` https://gerrit.wikimedia.org/g/mediawiki/extensions/Kartographer/+/2914baaa1260d7cf3cf9961fcb3e7503c028478e/modules/mapframe/mapframe.js#102 Is this still needed? If so could you provide context on what bug it is addressing so the #readers-growth team can consider whether to ? # TODO [X] Determine if Kartographer still needs it (And if there is a bug in production that has resurfaced relating to this) * alternative method found using InteractionObserver, and merged [X] If needed, update the documentation that Kartographer uses the hook and add it to the Parsoid sectionCollapsing code. * n/a as not needed [X] If not remove hook from code. * done

Many projects have been using parsoid now. While doing some code cleanup in MobileFrontend I noticed Kartographer makes use of the mobileFrontend.section-toggled hook which is not present in the Parsoid version and in the legacy parser marked as `Internal for use inside ExternalGuidance.` https://gerrit.wikimedia.org/g/mediawiki/extensions/Kartographer/+/2914baaa1260d7cf3cf9961fcb3e7503c028478e/modules/mapframe/mapframe.js#102 Is this still needed? If so could you provide context on what bug it is addressing so the #readers-growth team can consider whether to ? # TODO [X] Determine if Kartographer still needs it (And if there is a bug in production that has resurfaced relating to this) [ * alternative method found using InteractionObserver, and merged [X] If needed, update the documentation that Kartographer uses the hook and add it to the Parsoid sectionCollapsing code. * n/a as not needed [X] If not remove hook from code. * done [] If not remove hook from code.