Improve inline documentation (#41209)

WordPress · May 23, 2022 · 7af38a3 · 7af38a3
1 parent 5ad090d
commit 7af38a3
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/docs/reference-guides/data/data-core-block-editor.md b/docs/reference-guides/data/data-core-block-editor.md
@@ -1471,11 +1471,11 @@ _Parameters_
 
 ### setBlockVisibility
 
-Action that sets whether a block has controlled inner blocks.
+Action that sets whether given blocks are visible on the canvas.
 
 _Parameters_
 
--   _updates_ `Record<string,boolean>`: The block's clientId.
+-   _updates_ `Record<string,boolean>`: For each block's clientId, its new visibility setting.
 
 ### setHasControlledInnerBlocks
 

diff --git a/packages/block-editor/src/store/actions.js b/packages/block-editor/src/store/actions.js
@@ -1605,9 +1605,9 @@ export function setHasControlledInnerBlocks(
 }
 
 /**
- * Action that sets whether a block has controlled inner blocks.
+ * Action that sets whether given blocks are visible on the canvas.
  *
- * @param {Record<string,boolean>} updates The block's clientId.
+ * @param {Record<string,boolean>} updates For each block's clientId, its new visibility setting.
  */
 export function setBlockVisibility( updates ) {
 	return {

diff --git a/packages/components/src/autocomplete/index.js b/packages/components/src/autocomplete/index.js
@@ -269,6 +269,9 @@ function useAutocomplete( {
 		event.preventDefault();
 	}
 
+	// textContent is a primitive (string), memoizing is not strictly necessary
+	// but this is a preemptive performance improvement, since the autocompleter
+	// is a potential bottleneck for the editor type metric.
 	const textContent = useMemo( () => {
 		if ( isCollapsed( record ) ) {
 			return getTextContent( slice( record, 0 ) );