Details
As part of the loading indicator standardization, we are preparing to enable shouldUseGoBackButton by default in FullscreenLoadingIndicator. Before that, we need to migrate components where FullscreenLoadingIndicator is used alongside visible navigation to use ActivityIndicator instead, otherwise users would see two back controls (the header back button AND the Go Back button from the loader).
WorkspacePageWithSections is a shared layout component used by many workspace settings pages. It displays a HeaderWithBackButton (title passed via headerText) and uses a ternary to show either FullscreenLoadingIndicator or the main content when (isLoading || firstRender.current) && shouldShowLoading && isFocused and not offline. The loader is rendered with style={[styles.flex1, styles.pRelative]} in the content area below the header. The component is wrapped by withPolicyAndFullscreenLoading and rendered inside FullPageNotFoundView. Because this is a shared component, changes must be tested across all consuming pages.
Solution
Replace FullscreenLoadingIndicator with ActivityIndicator in src/pages/workspace/WorkspacePageWithSections.tsx. The replacement must be an ActivityIndicator wrapped in a parent View that provides the same layout and background styling as FullscreenLoadingIndicator. The loading state must be pixel-perfect identical after the change. Same background color, same opacity, same centered large spinner in the content area below the header.
Before making any changes, read the FullscreenLoadingIndicator component source to understand what styles it applies internally, and make sure the replacement reproduces them exactly. ActivityIndicator is a raw spinner with no layout or background styling, so it should not be used on its own as a fullscreen loading state. The codebase convention is to wrap it in a parent View that provides the layout and background styling.
Parent issue: #69850
Issue Owner
Current Issue Owner: @mateuuszzzzz
Details
As part of the loading indicator standardization, we are preparing to enable
shouldUseGoBackButtonby default inFullscreenLoadingIndicator. Before that, we need to migrate components whereFullscreenLoadingIndicatoris used alongside visible navigation to useActivityIndicatorinstead, otherwise users would see two back controls (the header back button AND the Go Back button from the loader).WorkspacePageWithSectionsis a shared layout component used by many workspace settings pages. It displays aHeaderWithBackButton(title passed viaheaderText) and uses a ternary to show eitherFullscreenLoadingIndicatoror the main content when(isLoading || firstRender.current) && shouldShowLoading && isFocusedand not offline. The loader is rendered withstyle={[styles.flex1, styles.pRelative]}in the content area below the header. The component is wrapped bywithPolicyAndFullscreenLoadingand rendered insideFullPageNotFoundView. Because this is a shared component, changes must be tested across all consuming pages.Solution
Replace
FullscreenLoadingIndicatorwithActivityIndicatorinsrc/pages/workspace/WorkspacePageWithSections.tsx. The replacement must be anActivityIndicatorwrapped in a parentViewthat provides the same layout and background styling asFullscreenLoadingIndicator. The loading state must be pixel-perfect identical after the change. Same background color, same opacity, same centered large spinner in the content area below the header.Before making any changes, read the
FullscreenLoadingIndicatorcomponent source to understand what styles it applies internally, and make sure the replacement reproduces them exactly.ActivityIndicatoris a raw spinner with no layout or background styling, so it should not be used on its own as a fullscreen loading state. The codebase convention is to wrap it in a parentViewthat provides the layout and background styling.Parent issue: #69850
Issue Owner
Current Issue Owner: @mateuuszzzzz