From ebd7b554deaf56245780e3123149225517ff3793 Mon Sep 17 00:00:00 2001
From: Cindy Qi Li <cli@ocadu.ca>
Date: Tue, 20 Dec 2022 14:11:16 -0500
Subject: [PATCH] GH-32: disable the page level reading

---
 src/js/enactors.js               | 36 +++++++++++---------------------
 tests/browser/js/enactorTests.js | 18 +++-------------
 2 files changed, 15 insertions(+), 39 deletions(-)

diff --git a/src/js/enactors.js b/src/js/enactors.js
index b151b57..20abe96 100644
--- a/src/js/enactors.js
+++ b/src/js/enactors.js
@@ -304,29 +304,17 @@ uioPlus.enactor.tableOfContents.injectToCContainer = function (that) {
 
 fluid.defaults("uioPlus.enactor.selfVoicing", {
     gradeNames: ["uioPlus.contentView", "fluid.prefs.enactor.selfVoicing"],
-    selectors: {
-        controllerParentContainer: ".flc-prefs-selfVoicingWidget",
-        domReaderContent: ".flc-orator-content"
-    },
-    domReaderContent: ["domReaderContent", "main", "article"],
-    controllerParentContainer: ["controllerParentContainer", "main", "article"],
-    distributeOptions: [{
-        record: {
-            expander: {
-                funcName: "uioPlus.contentView.findFirstSelector",
-                args: ["{selfVoicing}.locate", "{selfVoicing}.options.controllerParentContainer", "{selfVoicing}.container"]
-            }
+    // Temporarily disable the page level reading due to these issues:
+    // https://issues.fluidproject.org/browse/FLUID-6760
+    // https://issues.fluidproject.org/browse/FLUID-6761
+    distributeOptions: {
+        removeController: {
+            record: "fluid.emptySubcomponent",
+            target: "{that orator controller}.type"
         },
-        target: "{that orator > controller}.options.parentContainer",
-        namespace: "controllerParentContainer"
-    }, {
-        record: {
-            expander: {
-                funcName: "uioPlus.contentView.findFirstSelector",
-                args: ["{selfVoicing}.locate", "{selfVoicing}.options.domReaderContent", "{selfVoicing}.container"]
-            }
-        },
-        target: "{that orator}.options.components.domReader.container",
-        namespace: "domReaderContainer"
-    }]
+        removeDomReader: {
+            record: "fluid.emptySubcomponent",
+            target: "{that orator domReader}.type"
+        }
+    }
 });
diff --git a/tests/browser/js/enactorTests.js b/tests/browser/js/enactorTests.js
index 803d929..98ee48c 100644
--- a/tests/browser/js/enactorTests.js
+++ b/tests/browser/js/enactorTests.js
@@ -622,7 +622,7 @@ fluid.defaults("uioPlus.tests.selfVoicingTester", {
         name: "Self Voicing Tests",
         tests: [{
             name: "Model Changes",
-            expect: 5,
+            expect: 3,
             sequence: [{
                 // The orator subcomponent is not initialized until the enactor is enabled for the first time.
                 func: "uioPlus.tests.assertSubComponentNotCreated",
@@ -635,26 +635,14 @@ fluid.defaults("uioPlus.tests.selfVoicingTester", {
                 spec: {priority: "last:testing"},
                 listener: "uioPlus.tests.selfVoicingTester.assertOratorInit",
                 args: ["{selfVoicing}"]
-            }, {
-                func: "{selfVoicing}.applier.change",
-                args: ["enabled", false]
-            }, {
-                changeEvent: "{selfVoicing}.applier.modelChanged",
-                spec: {path: "enabled", priority: "last:testing"},
-                listener: "jqUnit.notVisible",
-                args: ["The orator controller should no longer be visible", "{selfVoicing}.orator.controller.container"]
             }]
         }]
     }]
 });
 
 uioPlus.tests.selfVoicingTester.assertOratorInit = function (that) {
-    var controller = that.orator.controller;
-    var domReader = that.orator.domReader;
-
-    jqUnit.assertTrue("The domReaders's container should be set properly", domReader.container.hasClass("flc-orator-content"));
-    jqUnit.assertTrue("The controller's parentContainer should be set properly", controller.options.parentContainer.hasClass("flc-prefs-selfVoicingWidget"));
-    jqUnit.isVisible("The orator controller should be visible", controller.container);
+    jqUnit.assertUndefined("The controller component is not instantiated", that.orator.controller);
+    jqUnit.assertUndefined("The domReaders component is not instantiated", that.orator.domReader);
 };
 
 fluid.test.runTests([