Simplify code comments for readability

KethanaReddy7 · KethanaReddy7 · commit ed38733af7f5 · 2026-04-02T13:06:44.000+05:30
diff --git a/src/scripts/clipperUI/components/sectionPicker.tsx b/src/scripts/clipperUI/components/sectionPicker.tsx
@@ -67,9 +67,7 @@ export class SectionPickerClass extends ComponentBase<SectionPickerState, Sectio
 		}
 		this.props.onPopupToggle(shouldNowBeOpen);
 		if (shouldNowBeOpen) {
-			// After the popup renders, move keyboard focus to the currently selected section item
-			// so that keyboard users can identify which item is selected and navigate from there.
-			// Also attach an arrow key handler so keyboard users can navigate the list with Up/Down.
+			// Focus the selected section and attach arrow key navigation for keyboard users
 			setTimeout(() => {
 				let sectionPickerPopup = document.getElementById("sectionPickerContainer");
 
@@ -86,21 +84,14 @@ export class SectionPickerClass extends ComponentBase<SectionPickerState, Sectio
 					elementToFocus.focus();
 				}
 
-				// Attach Up/Down arrow key navigation for the popup list.
-				// The OneNotePicker library only handles Enter/Tab, so we add arrow key support here.
-				// The listener is attached to the popup element which is removed from the DOM when the
-				// popup closes, so there is no need to explicitly clean it up.
-				// Guard against attaching multiple listeners if onPopupToggle(true) is called more than once.
-				// Use capture so the popup still receives Up/Down events before child controls suppress bubbling.
+				// Attach Up/Down arrow key navigation (OneNotePicker only handles Enter/Tab)
 				if (sectionPickerPopup && !sectionPickerPopup.getAttribute("data-arrow-key-handler-attached")) {
 					sectionPickerPopup.setAttribute("data-arrow-key-handler-attached", "true");
 					sectionPickerPopup.addEventListener("keydown", (e: KeyboardEvent) => {
 						if (e.which !== Constants.KeyCodes.up && e.which !== Constants.KeyCodes.down) {
 							return;
 						}
 						e.preventDefault();
-						// Only include visible items — exclude elements whose parent is inside a "Closed"
-						// collapsed notebook or sectionGroup (children remain in the DOM but are hidden via CSS).
 						let focusableItems = Array.prototype.slice.call(
 							sectionPickerPopup.querySelectorAll("[tabindex]:not([tabindex=\"-1\"])")
 						).filter((el) => {
@@ -302,7 +293,6 @@ export class SectionPickerClass extends ComponentBase<SectionPickerState, Sectio
 	}
 
 	// Attach escape key handler to return focus to the dropdown button when Escape is pressed
-	// This is needed because the OneNotePicker component handles Escape internally without calling onPopupToggle
 	attachEscapeFocusHandler(element: HTMLElement, isInitialized: boolean) {
 		if (!isInitialized) {
 			const escKeyCode = 27;
diff --git a/src/scripts/clipperUI/mainController.tsx b/src/scripts/clipperUI/mainController.tsx
@@ -101,11 +101,7 @@ export class MainControllerClass extends ComponentBase<MainControllerState, Main
 		}
 	}
 
-	/**
-	 * Handles focus trapping within certain panels to prevent keyboard focus from escaping
-	 * when navigating with Tab key. This ensures accessibility compliance by keeping focus
-	 * within the popup until the user explicitly closes it.
-	 */
+	// Prevents Tab key from moving focus outside the popup 
 	handleFocusTrap(event: KeyboardEvent) {
 		// Only trap focus for specific panels that need it
 		if (this.state.currentPanel !== PanelType.ClippingSuccess &&
diff --git a/src/scripts/extensions/clipperInject.ts b/src/scripts/extensions/clipperInject.ts
@@ -84,12 +84,7 @@ export class ClipperInject extends FrameInjectBase<ClipperInjectOptions> {
 			this.updateUiSizeAttributes();
 			this.overrideTransformStyles(document.documentElement);
 
-			// The clipper frame is now visible; hide page body from assistive technologies
-			// so that tools like Voice Access do not number background page controls.
-			// The frame is appended to <html> (not <body>), so marking <body aria-hidden>
-			// and inert only hides page content while keeping the clipper iframe accessible.
-			// aria-hidden removes body from the AT tree; inert also disables pointer events
-			// so Voice Access cannot enumerate or activate background controls.
+			// Hide page body from assistive technologies (Voice Access) while clipper is open
 			if (document.body) {
 				document.body.setAttribute("aria-hidden", "true");
 				document.body.setAttribute("inert", "");

Original file line number	Diff line number	Diff line change
`@@ -101,11 +101,7 @@ export class MainControllerClass extends ComponentBase<MainControllerState, Main`
`101`	`101`	`}`
`102`	`102`	`}`
`103`	`103`
`104`		`- /**`
`105`		`- * Handles focus trapping within certain panels to prevent keyboard focus from escaping`
`106`		`- * when navigating with Tab key. This ensures accessibility compliance by keeping focus`
`107`		`- * within the popup until the user explicitly closes it.`
`108`		`- */`
	`104`	`+ // Prevents Tab key from moving focus outside the popup`
`109`	`105`	`handleFocusTrap(event: KeyboardEvent) {`
`110`	`106`	`// Only trap focus for specific panels that need it`
`111`	`107`	`if (this.state.currentPanel !== PanelType.ClippingSuccess &&`