docs: Comments on the code

src/content-tags-drawer/ContentTagsDrawerHelper.jsx

@@ -339,6 +339,10 @@ const useContentTagsDrawerContext = (contentId) => {
 
     if (setBlockingSheet) {
       const areChangesInTags = () => {
+        // It is calculated in this way, because there are cases in which
+        // there are keys in the map, but they contain empty lists
+        // (e.g. add a tag, and remove the same tag later).
+
         const tagsAddedList = Object.values(globalStagedContentTags);
         const tagsRemovedList = Object.values(globalStagedRemovedContentTags);
 

src/content-tags-drawer/common/context.js

@@ -36,6 +36,11 @@ export const ContentTagsDrawerContext = React.createContext({
   setCollapsibleToInitalState: /** @type{() => void} */ (() => {}),
 });
 
+// This context has not been added to ContentTagsDrawerContext because it has been
+// created one level higher to control the behavior of the Sheet that contatins the Drawer.
+// This logic is not used in legacy edx-platform screens. But it can be separated if we keep
+// the contexts separate.
+// TODO We can join both contexts when the Drawer is no longer used on edx-platform
 /* istanbul ignore next */
 export const ContentTagsDrawerSheetContext = React.createContext({
   blockingSheet: /** @type{boolean} */ (false),