Add a comment on the arbitrary timeout

Author: tyxla

packages/edit-site/src/components/editor/index.js

@@ -164,6 +164,15 @@ export default function Editor() {
 	useEffect( () => {
 		if ( ! hasResolvingSelectors && ! loaded ) {
 			clearTimeout( timeoutRef.current );
+
+			/*
+			 * We're using an arbitrary 1s timeout here to catch brief moments
+			 * without any resolving selectors that would result in displaying
+			 * brief flickers of loading state and loaded state.
+			 *
+			 * It's worth experimenting with different values, since this also
+			 * adds 1s of artificial delay after loading has finished.
+			 */
 			timeoutRef.current = setTimeout( () => {
 				setLoaded( true );
 			}, 1000 );